-
Notifications
You must be signed in to change notification settings - Fork 20
Replace CommandBar's "More Items" button's "..." symbol with the Manage Tools symbol #3428
Conversation
|
I don't have a better suggestion but want to make sure we've thought about a few things. (Sorry if me not knowing how PI works means some of these don't make sense)
I think the other way to do this (has pros and cons) would be to put a flyout on a regular Primary button with the secondary items, then not have actual secondary items in the overflow. |
I'm concerned about this. Should all UI changes wait until the PMs settle on the UI they want? Also, what do you mean "What they were imagining?" Do they not have a mock up they can look at? |
Excellent questions. 1 - Yes, the primary reason is we're trying to more efficiently manage space on the bar. We have tools that have been pinned (primary commands) and tools that have not been pinned (secondary commands). If the user hasn't pinned anything to the command bar, we don't want the menu to consist of only a "..." item. Looks weird. At the same time, if folks are pinning multiple items, we don't want to waste a slot on the bar for a "manage tools" item in addition to the "...", as we're assuming these slots will be precious. The current implementation adds/removes the "managed tools" item from the primary commands based on the number of pinned tools, but we've also received feedback that buttons "randomly" appearing/disappearing isn't great either. 2- Yep. Current design has all of the tools show up in the context menu, and since I needed to remove labels from the command bar items (see #3425) this is the only way to see what the icons are for each pinned item. 3 - The current design turns off the dynamic overflow behavior so that no primary commands will show up in the overflow menu. Items will only show up once (as secondary commands) 4 - No other current plans, but obviously change can happen. Other way to do it - Yep, that would be the other way to do it (have a "Manage Tools" button show up in Command bar). See response to question #1. |
There is a mockup, but we're also exploring what's possible using "stock" WinUI3 controls, and we're still very much in an "iterate and learn" mode for Ironsides. |
Summary of the pull request
I've changed the CommandBar's "More Items" glyph from the ellipses to the "Manage tools" glyph
Old
New
I no longer am bouncing back and forth between having the "Manage Tools" action be either a primary command or a secondary command based on the number of pinned tools. It will always be on the screen.
I believe this is closer to what PM has been imagining.
References and relevant issues
Detailed description of the pull request / Additional comments
I hope someone looks at this, chuckles with a "well, that's one way to do it" and provides me a better way. From what the internet told me, I needed to duplicate CommandBar's ControlTemplate to just swap out the icon.
I don't think I can override the tooltip, but See More / See Less seemed ok to me for this scenario.
Validation steps performed
PR checklist