feat: change layer indicators from preset list #245
Conversation
✅ Deploy Preview for reearth-web ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Codecov Report
@@ Coverage Diff @@
## main #245 +/- ##
=======================================
Coverage 51.33% 51.33%
=======================================
Files 56 56
Lines 1165 1165
Branches 182 182
=======================================
Hits 598 598
Misses 505 505
Partials 62 62 |
…dicators-change-styles
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
src/components/molecules/Visualizer/Engine/Cesium/core/Indicator.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments.
Also, is there any chance we will support changing the color of the crosshair indicator? @lavalse maybe can shed some light on that. Currently the svg is setup to only be white, but would be pretty simple to allow for color.
…dicators-change-styles
…/reearth-web into indicators-change-styles
…or.tsx Co-authored-by: KaWaite <34051327+KaWaite@users.noreply.github.com>
Co-authored-by: KaWaite <34051327+KaWaite@users.noreply.github.com>
…or.tsx Co-authored-by: KaWaite <34051327+KaWaite@users.noreply.github.com>
…or.tsx Co-authored-by: KaWaite <34051327+KaWaite@users.noreply.github.com>
…dicators-change-styles
…dicators-change-styles
Sorry for testing a little bit late. But I found some problems maybe more importent then animation. 😄
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…dicators-change-styles
Overview
Layer selecting indicator: is an indicator appears when user select a layer.
We created preset list of indicators that user can choose one of them when selecting a layer.
What I've done
Design a preset list of indicators for layer selecting.
Add indicator section in scene right tab and give the user the ability to choose among them.
Give the user the ability to upload his own indicator.
What I haven't done
How I tested
Screenshot
Which point I want you to review particularly
Memo
Design link : https://www.figma.com/file/bdnfDaGXGoNfUws4y7JItl/Re%3AEarth-UI-Gamma?node-id=10319%3A174052
Backend PR: reearth/reearth-backend#158