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

Support dark mode in maps-dashboards #455

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

junqiu-lei
Copy link
Member

@junqiu-lei junqiu-lei commented Aug 16, 2023

Description

Support dark mode in maps-dashboards:

  • Introduce dark style base tile map to be applied automatically from OSD style mode
  • Update Draw policy icon to support dark mode
  • Update map controller(zoom in/zoom out/compass) to support dark mode
  • Update opensearchVectorTileStyleUrl to be a style manifest which can contain multiple styles from server.

Issues Resolved

#448

Demo

Screen.Recording.2023-08-15.at.11.55.18.PM.mov

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@codecov
Copy link

codecov bot commented Aug 16, 2023

Codecov Report

Merging #455 (317eadc) into main (a4e92a0) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #455      +/-   ##
==========================================
- Coverage   41.40%   41.36%   -0.05%     
==========================================
  Files          53       53              
  Lines        1775     1789      +14     
  Branches      326      328       +2     
==========================================
+ Hits          735      740       +5     
- Misses       1012     1019       +7     
- Partials       28       30       +2     

see 9 files with indirect coverage changes

@junqiu-lei junqiu-lei requested a review from KrooshalUX August 16, 2023 18:30
Signed-off-by: Junqiu Lei <junqiu@amazon.com>
@junqiu-lei
Copy link
Member Author

Cypress tests passed at local, the Github cypress CI failure is unrelated to this PR.

@junqiu-lei junqiu-lei merged commit 5344f27 into opensearch-project:main Aug 18, 2023
@junqiu-lei junqiu-lei deleted the dark1 branch August 18, 2023 02:53
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 18, 2023
Signed-off-by: Junqiu Lei <junqiu@amazon.com>
(cherry picked from commit 5344f27)
junqiu-lei added a commit that referenced this pull request Aug 18, 2023
Signed-off-by: Junqiu Lei <junqiu@amazon.com>
(cherry picked from commit 5344f27)

Co-authored-by: Junqiu Lei <junqiu@amazon.com>
@junqiu-lei junqiu-lei mentioned this pull request Aug 18, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x feature v2.10.0 Issues targeting release v2.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants