Skip to content

Change text highlight color so we can see the text#170

Merged
Leemoonsoo merged 1 commit intomasterfrom
fix/TextHighlightColor
Oct 23, 2014
Merged

Change text highlight color so we can see the text#170
Leemoonsoo merged 1 commit intomasterfrom
fix/TextHighlightColor

Conversation

@corneadoug
Copy link
Copy Markdown
Contributor

When we highlighted the text, we couldn't see the text since background-color was applied on top of it.
This resolve the problem by adding some alpha to the color

@Leemoonsoo
Copy link
Copy Markdown
Contributor

I have tested and all looks good! can't wait.

@corneadoug
Copy link
Copy Markdown
Contributor Author

Let's Merge it then since CI build passed :)

Leemoonsoo added a commit that referenced this pull request Oct 23, 2014
Change text highlight color so we can see the text
@Leemoonsoo Leemoonsoo merged commit fe33faf into master Oct 23, 2014
@Leemoonsoo Leemoonsoo deleted the fix/TextHighlightColor branch October 23, 2014 03:38
epahomov pushed a commit to epahomov/zeppelin that referenced this pull request Jul 23, 2016
This PR fixes https://issues.apache.org/jira/browse/ZEPPELIN-172

Author: Lee moon soo <[email protected]>
Author: Sjoerd Mulder <[email protected]>
Author: Lee moon soo <[email protected]>

Closes ZEPL#170 from Leemoonsoo/websocket and squashes the following commits:

11a302a [Lee moon soo] Check text in more safe way
3cf839d [Lee moon soo] Merge pull request ZEPL#2 from sjoerdmulder/websocket
7f8bc47 [Sjoerd Mulder] Cleanup of Javascript logic and Server code detecting the correct port
412927f [Lee moon soo] Handle large message
f56e417 [Lee moon soo] Add license header
806db9b [Lee moon soo] Remove websocket addr/port configuration
6180ed3 [Lee moon soo] Update README
85d14a0 [Lee moon soo] Create notebookserver instance manually
a7b82aa [Lee moon soo] Initial implementation of Websocket inside of Jetty server
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants