EQWorks / widget-studio

Widget-creating tool for charts, maps, tables, stats
https://eqworks.github.io/widget-studio/
2 stars 3 forks source link

[G2M] Add aliases for column titles #186

Closed geoerika closed 1 year ago

geoerika commented 1 year ago

Closes #185

Changes:

ColumnAliasControls - for devs only

Test Story

Screen Shot 2022-10-25 at 10 16 42 AM

Recording:

https://user-images.githubusercontent.com/41120953/197799479-47c8efc0-1278-440d-ac71-8b818646b0b9.mov

github-actions[bot] commented 1 year ago

📚 Storybook preview (updated to 6c3ca8601b2572b45f20389d23994e1c0741f52f)

geoerika commented 1 year ago

@Hamaad102

The domain configuration lacks the icons for column and alias

Yes, I know. But that's how the design for that section looks like. We can revisit this later. Widget Studio needs a full revision for styling, something we have been pushing aside since the design team could not fit it in their schedule. From Figma:

Screen Shot 2022-10-26 at 9 51 55 AM
geoerika commented 1 year ago

@Hamaad102

Textfield is slightly finicky. If you make changes and immediately hit the done button it most likely will not save. If you wait a moment or hit the enter button the changes will save. I can provide video of this if needed.

No need for the recording, I am using a debounced value to update the state so I don't have too frequent re-renders in the charts/map. I just reduced the time in useDebounce from 1 sec to .5 sec. Please try it again and let me know if I need to decrease the time further.

Hamaad102 commented 1 year ago

@Hamaad102

Textfield is slightly finicky. If you make changes and immediately hit the done button it most likely will not save. If you wait a moment or hit the enter button the changes will save. I can provide video of this if needed.

No need for the recording, I am using a debounced value to update the state so I don't have too frequent re-renders in the charts/map. I just reduced the time in useDebounce from 1 sec to .5 sec. Please try it again and let me know if I need to decrease the time further.

I just tested it, this adjustment is perfect

geoerika commented 1 year ago

@Hamaad102 I also fixed all the other issues you pointed out, if you can re-review please!

Hamaad102 commented 1 year ago

Screenshot 2022-10-26 102459

When you click the done button and go back to the normal view the add value button is also temporarily missing. Pressing clear once will actually clear the alias. Pressing it a second time will display the add value button. I think in this instance it should not be there at all? This is for dev-bar-2 specifically where normally the add value button is not there at all. Screenshot 2022-10-26 102807

Otherwise, everything looks good

vlatawiec commented 1 year ago

@Hamaad102

The domain configuration lacks the icons for column and alias

Yes, I know. But that's how the design for that section looks like. We can revisit this later. Widget Studio needs a full revision for styling, something we have been pushing aside since the design team could not fit it in their schedule. From Figma: Screen Shot 2022-10-26 at 9 51 55 AM

This is a definite addition for consistency. I can create a design for this (if it's a quick/simple iteration for DEV) asap, otherwise let me know when this becomes a priority again!

geoerika commented 1 year ago

This is a definite addition for consistency. I can create a design for this (if it's a quick/simple iteration for DEV) asap, otherwise let me know when this becomes a priority again!

@vlatawiec if you have time, sure, go ahead. I will take care of it when I get a window.

geoerika commented 1 year ago

@vlatawiec thank you! @Hamaad102 I fixed the last issues. Good catches! Thanks!