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

Reversed order of layer toggles #658

Merged
merged 4 commits into from
Nov 24, 2021
Merged

Reversed order of layer toggles #658

merged 4 commits into from
Nov 24, 2021

Conversation

hkfb
Copy link
Collaborator

@hkfb hkfb commented Nov 22, 2021

Represents the deck.gl layer stack more intuitively.

Fixes #620

Havard Bjerke added 2 commits November 22, 2021 10:31
Represents the deck.gl layer stack more intuitively.

Fixes equinor#620
@hkfb hkfb added AspenTech Task owned by AspenTech map-component Issues related to the map component. labels Nov 22, 2021
@hkfb hkfb self-assigned this Nov 22, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 22, 2021

Codecov Report

Merging #658 (ccc37db) into master (fa9ff01) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #658   +/-   ##
=======================================
  Coverage   73.46%   73.46%           
=======================================
  Files          58       58           
  Lines        1176     1176           
  Branches      153      153           
=======================================
  Hits          864      864           
  Misses        294      294           
  Partials       18       18           
Impacted Files Coverage Δ
...nts/DeckGLMap/components/settings/LayersButton.tsx 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa9ff01...ccc37db. Read the comment docs.

Copy link
Contributor

@shadab-skhan shadab-skhan left a comment

Choose a reason for hiding this comment

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

This works, but still the layer ordering is governed by the order in which it's defined in deckgl-map.json file. Do we always want layers to be in particular fixed order? If that being the case, we need to do some more changes.
Or for storybook we can change the ordering in deckgl-map.json file instead of dong it here.

@hkfb
Copy link
Collaborator Author

hkfb commented Nov 23, 2021

This works, but still the layer ordering is governed by the order in which it's defined in deckgl-map.json file. Do we always want layers to be in particular fixed order? If that being the case, we need to do some more changes. Or for storybook we can change the ordering in deckgl-map.json file instead of dong it here.

The purpose is only to change to a more intuitive representation in the layer selector. This does not and should not affect representation when drawing layers.

@shadab-skhan shadab-skhan self-requested a review November 24, 2021 03:50
@hkfb hkfb merged commit 9a8f896 into equinor:master Nov 24, 2021
@hkfb hkfb deleted the reverse_toggles branch November 24, 2021 07:21
hkfb added a commit to hkfb/webviz-subsurface-components that referenced this pull request Nov 29, 2021
* Reversed order of layer toggles.

Represents the deck.gl layer stack more intuitively.

Fixes equinor#620

* Applied lint conditions.

Co-authored-by: Havard Bjerke <[email protected]>
Co-authored-by: Shadab Khan <[email protected]>
hkfb pushed a commit that referenced this pull request Jul 30, 2024
hkfb pushed a commit that referenced this pull request Jul 30, 2024
hkfb pushed a commit that referenced this pull request Jul 30, 2024
hkfb pushed a commit that referenced this pull request Jul 30, 2024
hkfb pushed a commit that referenced this pull request Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AspenTech Task owned by AspenTech map-component Issues related to the map component.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Reverse order of layer toggles in toolbox
3 participants