search

BCDA-APS / gemviz

Data visualization for tiled
https://bcda-aps.github.io/gemviz/
Other
4 stars 0 forks source link

add refresh button (to re-apply filters) #107

Closed prjemian closed 1 year ago

prjemian commented 1 year ago
prjemian commented 1 year ago

refresh button is better placed in the date time range slider widget. to the right of the slider

prjemian commented 1 year ago

The refresh button image from moxy looks too small and blurry. We'll work up a replacement.

prjemian commented 1 year ago

@rodolakis This is not ready for review but you said you could offer a different image for the refresh button.

prjemian commented 1 year ago

Here's a view with the current button image (displayed at full 30x30 size): image

prjemian commented 1 year ago

I believe the image formats for the QPushButton are flexible. See this web search for xpm & bmp formats. Likely others are possible.

prjemian commented 1 year ago

@rodolakis All this PR needs is some new art for the refresh button. As I look at it now, the current artwork suggest that the button might reset the date/time values and slider. We want it to look like it will apply any and all the search terms in the panel.

prjemian commented 1 year ago

Since all that this PR needs is new artwork for the button, I moved that to a separate issue. After this PR, I plan to work on #113 which must be done while no other branches exist (or there will be merge conflicts).