Skip to content

resolves #128 : add inverted color option on trends panel#935

Closed
chocobn69 wants to merge 2 commits intoelastic:masterfrom
chocobn69:#128
Closed

resolves #128 : add inverted color option on trends panel#935
chocobn69 wants to merge 2 commits intoelastic:masterfrom
chocobn69:#128

Conversation

@chocobn69
Copy link
Copy Markdown

No description provided.

src/config.js Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Don't think this changes was intended?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

oh god sorry, didn't check before submit

can you remove this change ? or do I have to resubmit a new pull request ?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You can do a new commit on the same branch you submitted this patch. then it will automatically be in this PR.

@chocobn69
Copy link
Copy Markdown
Author

oh god sorry, didn't check before submit

@chocobn69
Copy link
Copy Markdown
Author

should be ok now

@electrical
Copy link
Copy Markdown

Yeah, perfect :-)
Im sure @rashidkpc will have a look at this soon(tm)

@rashidkpc
Copy link
Copy Markdown
Contributor

Looks good, can you make sure you've signed the CLA mentioned in the CONTRIBUTING.md?

@chocobn69
Copy link
Copy Markdown
Author

I just did

@rashidkpc
Copy link
Copy Markdown
Contributor

Closed here: 00bd2c8

@rashidkpc rashidkpc closed this Apr 28, 2014
@chocobn69 chocobn69 deleted the #128 branch April 29, 2014 06:21
edmarmoretti pushed a commit to edmarmoretti/kibana-pt-br that referenced this pull request Sep 7, 2025
edmarmoretti pushed a commit to edmarmoretti/kibana-pt-br that referenced this pull request Sep 7, 2025
# [35.0.0](elastic/elastic-charts@v34.2.1...v35.0.0) (2021-09-13)

### Bug Fixes

* **a11y:** restore focus after popover close with color picker ([elastic#1272](elastic/elastic-charts#1272)) ([0c6f945](elastic/elastic-charts@0c6f945)), closes [elastic#1266](elastic/elastic-charts#1266) [elastic#935](elastic/elastic-charts#935)
* **build:** fix license in package.json ([elastic#1362](elastic/elastic-charts#1362)) ([d524fdf](elastic/elastic-charts@d524fdf))
* **deps:** update dependency @elastic/eui to ^37.5.0 ([elastic#1341](elastic/elastic-charts#1341)) ([fb05c98](elastic/elastic-charts@fb05c98))
* **deps:** update dependency @elastic/eui to ^37.6.1 ([elastic#1359](elastic/elastic-charts#1359)) ([2ae90ce](elastic/elastic-charts@2ae90ce))
* **deps:** update dependency @elastic/eui to ^37.7.0 ([elastic#1373](elastic/elastic-charts#1373)) ([553b6b0](elastic/elastic-charts@553b6b0))
* **heatmap:** filter out tooltip picked shapes in x-axis area ([elastic#1351](elastic/elastic-charts#1351)) ([174047d](elastic/elastic-charts@174047d)), closes [elastic#1215](elastic/elastic-charts#1215)
* **heatmap:** remove values when brushing only over axes ([elastic#1364](elastic/elastic-charts#1364)) ([77ff8d3](elastic/elastic-charts@77ff8d3))

### Features

* **annotations:** add onClickHandler for annotations ([elastic#1293](elastic/elastic-charts#1293)) ([48198be](elastic/elastic-charts@48198be)), closes [elastic#1211](elastic/elastic-charts#1211)
* **heatmap:** add text color contrast to heatmap cells ([elastic#1342](elastic/elastic-charts#1342)) ([f9a26ef](elastic/elastic-charts@f9a26ef)), closes [elastic#1296](elastic/elastic-charts#1296)
* **heatmap:** reduce font size to fit label within cells ([elastic#1352](elastic/elastic-charts#1352)) ([16b5546](elastic/elastic-charts@16b5546))
* **xy:** mutilayer time axis step 1 ([elastic#1326](elastic/elastic-charts#1326)) ([867b1f5](elastic/elastic-charts@867b1f5))

### BREAKING CHANGES

* **xy:** - feat: removes the axis deduplication feature
- fix: `showDuplicatedTicks` causes a duplication check on the actual axis tick label (possibly yielded by `Axis.tickLabel` rather than the more general `tickFormat`)
* **heatmap:** the `config.label.fontSize` prop is replaced by `config.label.minFontSize` and `config.label.maxFontSize`. You can specify the same value for both properties to have a fixed font size. The `config.label.align` and `config.label.baseline` props are removed from the `HeatmapConfig` object.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants