Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improves timelion axis and legend labels color with dark theme #9422

Merged
merged 1 commit into from
Dec 16, 2016

Conversation

ppisljar
Copy link
Member

@ppisljar ppisljar commented Dec 9, 2016

Improves axis and legend label colors in timelion with dark theme enabled on dashboard.

fixes #9362

screenshot-localhost 5601 2016-12-12 09-13-38

@ppisljar ppisljar added Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v5.2.0 v6.0.0 labels Dec 9, 2016
@tbragin
Copy link
Contributor

tbragin commented Dec 11, 2016

@ppisljar What about legends? Their color is also too dark. Do you plan to follow up with another PR to do that?

@ppisljar ppisljar force-pushed the fix/timelion/axisColor branch from d53f77e to 018105d Compare December 12, 2016 08:11
@ppisljar ppisljar changed the title improves timelion axis labels color with dark theme improves timelion axis and legend labels color with dark theme Dec 12, 2016
@ppisljar
Copy link
Member Author

sorry, both are updated in this PR

@thomasneirynck thomasneirynck self-assigned this Dec 13, 2016
Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, both legend and gridlines.

@tbragin
Copy link
Contributor

tbragin commented Dec 15, 2016

@ppisljar Looks good! LGTM

screen shot 2016-12-15 at 3 22 54 pm

@ppisljar ppisljar merged commit 30f0c6f into elastic:master Dec 16, 2016
elastic-jasper added a commit that referenced this pull request Dec 16, 2016
Backports PR #9422

**Commit 1:**
pie chart unhandled error fix

* Original sha: 018105d
* Authored by ppisljar <[email protected]> on 2016-12-09T11:02:39Z
ppisljar pushed a commit that referenced this pull request Dec 16, 2016
Backports PR #9422

**Commit 1:**
pie chart unhandled error fix

* Original sha: 018105d
* Authored by ppisljar <[email protected]> on 2016-12-09T11:02:39Z
stacey-gammon added a commit to stacey-gammon/kibana that referenced this pull request Jan 19, 2017
clean up

show spy panel in embed mode

[webpack] fix loader query string usage (elastic#9497)

* [webpack] pin to fork with fixed loader aliases

* [optimizer] upgrade to postcss+autoprefixer

* [timelion] convert uiExports.modules -> webpackShims

* [uiExports] remove implementation-leaking and unused uiExport types

* [optimizer] remove unused imports

* [uiBundlerEnv] add a method for exporting global import aliases for special cases

[dev tools] Hide app link when there are no tools (elastic#9489)

* [dev tools] Hide app link when there are no tools

* [dev tools] Add tests for setting app as hidden

pie chart unhandled error fix (elastic#9422)

Add NoResults and Panel components. (elastic#9516)

* Add NoResults and Panel components.

* Lighten noResults text.

Update ToolBarFooter component to support content on the left side. (elastic#9514)

Fix bug with Button component appearance inside of a ToolBar. (elastic#9526)

Make basic Button hover state the same both in and out of ToolBar. (elastic#9528)

[grunt/eslint] fix precommit linting (elastic#9510)

* [grunt/eslint] fix precommit linting

 - remove use of `minimatch.makeRe()` because it does not support the entire glob syntax
 - log a warning whenever a js file is excluded by the `lintStagedFiles` task
 - eslint globs are relative to the project root, ensure that we check against relative version

* [grunt/eslint] only log warning wtr grunt paths

Add Tabs component. (elastic#9536)

- Fix bugs with Button and CheckBox focused states.
- Fix appearance of cell content in Table.

Disable linting for Tabs component example JS. (elastic#9538)

Set Button component to display: inline-block, to ensure it has the same height when applied to both button elements and anchor tags. (elastic#9541)

fixing metric vis to correctly show scrollbars when overflown (elastic#9481)

Adding Safari 7 support to autoprefixer (elastic#9534)

PhantomJS is using a rather outdated version of WebKit, which requires
various css-prefixes to render correctly. PhantomJS doesn't have a specific
user-agent, and Safari 7 is the closet version of WebKit.

use Stop Editing instead of Preview

Warn the user if they Stop Editing with unsaved changes

- Refresh the dashboard after stop editing so unsaved changes are lost
and no temporary edits will be shown in non-edit mode.

Don't watch the variable on scope, but the config attribute
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v5.2.0 v6.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Timelion legends and axis values in dark theme almost unreadable
3 participants