Solar Sites UI MVP => nowcasting UI for individual pv site owners #338
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
|
||
| const SitesMapSlider: React.FC<{ selected?: string; unselected?: string }> = ({ | ||
| const SitesMapSlider: React.FC<{ selected?: string; unselected?: string; event?: any }> = ({ |
There was a problem hiding this comment.
Hey @braddf, This is what I came up with for the initial "slider" for the map. Would be great to have feedback on the ranges for the zoom levels (line 13-22) and if this is sort of what you were thinking. I know it isn't a traditional "slider". I felt like keeping the look boxy fit with the design. When you click the +/- buttons, the highlighted view changes.
As you can see, I tried adding an event handler to each div, but Typescript seemed unhappy with the event.target situation when I pushed my code. That's why it's been deleted. Didn't seem like it was necessary for this commit.
Any other feedback, pointers are welcome.
There was a problem hiding this comment.
Great - just making sure I've been through all these comments, sorry for the delay... we can take a look at this design on the next iteration when we finish the aggregation stuff; I have some ideas to make this a little less intrusive, but we'll have to play around and see what feels right 👍
|
|
||
| const SitesMapSlider: React.FC<{ selected?: string; unselected?: string }> = ({ | ||
| const SitesMapSlider: React.FC<{ selected?: string; unselected?: string; event?: any }> = ({ |
There was a problem hiding this comment.
Great - just making sure I've been through all these comments, sorry for the delay... we can take a look at this design on the next iteration when we finish the aggregation stuff; I have some ideas to make this a little less intrusive, but we'll have to play around and see what feels right 👍
Solar Sites UI MVP
The work here responds to #323 that will be building the following feature:
Like with #287, the
delta viewfeature, we'll be working on thefeat/323-solar-sitesbranch and using this PR to communicate about and review code as the work progresses. If it seems necessary to make branches off of this branch for some components, such as map features, then we shall do so.Description
This is the individual PV site view that offers a UI to individual pv site owners with a specific pv forecast for each of their sites.
Builds #323
How Has This Been Tested?
Code will be tested by running it locally and inspecting UI/UX on the deployed Vercel branches.
Checklist: