Skip to content

[Maps] Reverse the attribution order so left-to-right order matches top-to-bottom#44415

Merged
kindsun merged 3 commits intoelastic:masterfrom
kindsun:reverse-attribution-order
Sep 3, 2019
Merged

[Maps] Reverse the attribution order so left-to-right order matches top-to-bottom#44415
kindsun merged 3 commits intoelastic:masterfrom
kindsun:reverse-attribution-order

Conversation

@kindsun
Copy link
Copy Markdown
Contributor

@kindsun kindsun commented Aug 29, 2019

Small follow-up PR to #44147 to match the left-to-right attribution order to top-to-bottom layer order on the TOC and in the map.

@kindsun kindsun added release_note:fix Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 v7.4.0 v7.5.0 labels Aug 29, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-gis

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@kindsun kindsun force-pushed the reverse-attribution-order branch from 1214477 to 32d1432 Compare August 30, 2019 19:44
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@kindsun kindsun requested a review from nickpeihl August 30, 2019 21:37
Copy link
Copy Markdown
Contributor

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

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

lgtm

tested in chrome and code review

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@kindsun kindsun merged commit 4cbe9ff into elastic:master Sep 3, 2019
kindsun pushed a commit to kindsun/kibana that referenced this pull request Sep 3, 2019
…op-to-bottom (elastic#44415)

* Reverse the attribution order so left-to-right order matches top-to-bottom layer order

* Update snapshot to reflect new attribution order
kindsun pushed a commit to kindsun/kibana that referenced this pull request Sep 3, 2019
…op-to-bottom (elastic#44415)

* Reverse the attribution order so left-to-right order matches top-to-bottom layer order

* Update snapshot to reflect new attribution order
jloleysens added a commit to jloleysens/kibana that referenced this pull request Sep 4, 2019
…-to-np-ready

* 'master' of github.com:elastic/kibana: (45 commits)
  Fixing flaky copy-to-space test (elastic#44656)
  [ML] Enable data visualizer link on job types page for non-time index (elastic#44667)
  [ML] Adding index pattern title to wizard summary step (elastic#44662)
  Convert remaining data.indexPatterns items to use TS (elastic#44654)
  [visualize/_tsvb_time_series] fix flaky test (elastic#44505)
  [SIEM] Updates fields in map tooltip and fixes filtering on undefined values (elastic#44718)
  [SIEM] Remove deprecation warning from momentJS test (elastic#44732)
  [SIEM] Fixes the rendering order and adds unit tests to prevent rendering to get out of order (elastic#44728)
  [optimizer] use more precise cache-busting strategy (elastic#44724)
  [Maps] Reverse the attribution order so left-to-right order matches top-to-bottom (elastic#44415)
  Update sinon related packages (elastic#43952)
  [SIEM] change flag to drag and drop field (elastic#44705)
  Update dependency @types/object-hash to ^1.3.0 (elastic#43954)
  mark eslint config as root (elastic#44699)
  Adds geo_point to normalizeType helper (elastic#44410)
  [Uptime]Feature/38542  align uptime help menu with other apps (elastic#44536)
  Removed unused remove icon component (elastic#44698)
  Update unit tests to remove warnings from test output. (elastic#41840)
  Fix rawChecksums in chromium build and rename for clarity (elastic#44665)
  [Uptime]removed extra slash from integeration urls (elastic#44469)
  ...

# Conflicts:
#	rfcs/text/0004_application_service_mounting.md
jloleysens added a commit to jloleysens/kibana that referenced this pull request Sep 4, 2019
…ana into chore/console-to-np-ready

* 'chore/console-to-np-ready' of github.com:jloleysens/kibana: (45 commits)
  Fixing flaky copy-to-space test (elastic#44656)
  [ML] Enable data visualizer link on job types page for non-time index (elastic#44667)
  [ML] Adding index pattern title to wizard summary step (elastic#44662)
  Convert remaining data.indexPatterns items to use TS (elastic#44654)
  [visualize/_tsvb_time_series] fix flaky test (elastic#44505)
  [SIEM] Updates fields in map tooltip and fixes filtering on undefined values (elastic#44718)
  [SIEM] Remove deprecation warning from momentJS test (elastic#44732)
  [SIEM] Fixes the rendering order and adds unit tests to prevent rendering to get out of order (elastic#44728)
  [optimizer] use more precise cache-busting strategy (elastic#44724)
  [Maps] Reverse the attribution order so left-to-right order matches top-to-bottom (elastic#44415)
  Update sinon related packages (elastic#43952)
  [SIEM] change flag to drag and drop field (elastic#44705)
  Update dependency @types/object-hash to ^1.3.0 (elastic#43954)
  mark eslint config as root (elastic#44699)
  Adds geo_point to normalizeType helper (elastic#44410)
  [Uptime]Feature/38542  align uptime help menu with other apps (elastic#44536)
  Removed unused remove icon component (elastic#44698)
  Update unit tests to remove warnings from test output. (elastic#41840)
  Fix rawChecksums in chromium build and rename for clarity (elastic#44665)
  [Uptime]removed extra slash from integeration urls (elastic#44469)
  ...
kindsun pushed a commit that referenced this pull request Sep 5, 2019
…op-to-bottom (#44415) (#44726)

* Reverse the attribution order so left-to-right order matches top-to-bottom layer order

* Update snapshot to reflect new attribution order
kindsun pushed a commit that referenced this pull request Sep 5, 2019
…op-to-bottom (#44415) (#44727)

* Reverse the attribution order so left-to-right order matches top-to-bottom layer order

* Update snapshot to reflect new attribution order
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:fix Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.4.0 v7.5.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants