Add menu dropdown and status button #41
Conversation
| html.Div( | ||
| [ | ||
| settings.open_btn | ||
| dbc.Button( |
There was a problem hiding this comment.
I would add these are items in the dropdown menu. That way the menu contains more than just logout. You may have to alter the settings.open_btn to address this. The settings open button also includes a cog (settings icon), that can probably be removed if its moved into the dropdown. There is a classname on the text of the open settings button that hides when the screen is small, this will need to be addressed as well.
There was a problem hiding this comment.
@bradley-erickson I'll be pushing it in a bit, but first, I want to advice that we shouldn't put the "last updated" status in the dropdown too for consistency sake since it is not a actionable/clickable button.
I have added the settings toggle button into the dropdown, if you give me a go ahead, I will just update my PR with that
There was a problem hiding this comment.
I've updated it
There was a problem hiding this comment.
Can you wrap the button and the dropdown menu in a button group and include a picture with it as well?
See the dropdown section here.
Using outlines instead of full color buttons might look a little nicer as well
There was a problem hiding this comment.
I have wrapped it in the button group, but what do you mean by "include a picture with it as well"?
There was a problem hiding this comment.
Just take a screenshot. and include it here. That way we and whoever else looks at this PR can quickly see what's changed.
No description provided.