l10nelw / winger

Window Manager: A Firefox web extension for switching windows, moving tabs between windows, and more
https://addons.mozilla.org/firefox/addon/winger/
GNU General Public License v3.0
48 stars 7 forks source link

Submitting new window name after /edit (or /name) no longer works #16

Closed mkalinski closed 1 year ago

mkalinski commented 1 year ago

To reproduce

  1. Click on the Winger extension button.
  2. Type /edit or /name and press enter.
  3. The prompt to rename the window opens.
  4. Type any name and press enter.

What happens

Nothing. The new name is not submitted, the prompt remains open and active, but the edit cannot be completed in any way other than exiting out from the extension menu (which cancels the edit).

What should happen (and did in the previous version of Winger)

The name is submitted as the new name of the window, and we go back to the normal search box that's active when the extension menu is opened.


Note that the expected behaviour happens when the name editing prompt is activated by clicking on the active window's name. It's only when it's accessed by typing the slash command that it doesn't work.

l10nelw commented 1 year ago

When entering edit mode via /edit or /name, the omnibox stays targeted, so you need to press Up/Down keys or click to target the row you want. Typing in the omnibox performs filtering as usual, which now also works in edit mode.

It's actually easier to rename the current window now; from the omnibox you just press Up then Enter to activate edit mode (current window targeted).

Thank you for sharing your issue, I'll see if I could design things to be much clearer, and clarify in the help as well.

mkalinski commented 1 year ago

Okay. I admit, I didn't quite understand what was meant when the docs mentioned "filtering in edit mode", but I didn't really care, since I couldn't think of a use case I would have for this.

I think the tooltip is terse to the point of being slightly misleading.

image

Yes, it says "on a name", but there's no indication I could move to a name. And from my point of view, I've just typed a name.

But the core of the issue probably is that doing the same thing that worked in version 1.x doesn't anymore.

Perhaps it would be good to display clarification on the tooltip when pressing enter in the omnibox (since it seems to do nothing currently)?

l10nelw commented 1 year ago

Yes a better message in the status bar for this condition is in order. Thanks.

I couldn't think of a use case I would have for this

When you have a large number of windows, filtering would be helpful to find the window you want.

l10nelw commented 1 year ago

Relevant 2.0.1 updates for edit mode: