Marco Zehe (:MarcoZ) Reporter
	Description • 10 years ago

Currently, the Hello button to start a WebRTC phone call is not accessible via the keyboard. There is only a button in the tool bar, but there is no menu equivalent in the classic menu or a keyboard shortcut, or if there is a keyboard shortcut, it is not discoverable.

	:Gijs (he/him) Assignee
	Comment 1 • 10 years ago

Maire, can this be prioritized? For sec508 and various other reasons we shouldn't really ship major features without keyboard a11y...

	Mike de Boer [:mikedeboer]
	Comment 3 • 9 years ago

We should
1) Add a menu item to the 'Tools' main menu (this location is my personal suggestion)
2) Add a direct keyboard shortcut to open the Loop panel, just like we have one for the Menu Panel.

There's a bit of discussion necessary about which shortcut to use and what the menu item label should read.

	:Gijs (he/him) Assignee
	Comment 4 • 9 years ago

(In reply to Mike de Boer [:mikedeboer] from comment #3)
> We should
> 1) Add a menu item to the 'Tools' main menu (this location is my personal
> suggestion)
> 2) Add a direct keyboard shortcut to open the Loop panel, just like we have
> one for the Menu Panel.

There isn't a shortcut for the menu panel anymore, that was backed out.

> There's a bit of discussion necessary about which shortcut to use and what
> the menu item label should read.

	Matej Novak [:matej]
	Comment 5 • 9 years ago

I'll look to Sevaan for UX best practices in this case, but here are some options for what it could say:

Start a conversation

Open Hello

Start Hello

New Hello window

New Hello conversation

New video conversation

	(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo
	Comment 6 • 9 years ago

Mike are you suggesting to add just a menu item to open the Hello panel, or actually mirroring some of the functionality in a sub-menu.

I'd prefer the former, but the Hello panel itself is not very keyboard-friendly itself.

	Mike de Boer [:mikedeboer]
	Comment 7 • 9 years ago

(In reply to Philipp Sackl [:phlsa] from comment #6)
> Mike are you suggesting to add just a menu item to open the Hello panel, or
> actually mirroring some of the functionality in a sub-menu.
> 
> I'd prefer the former, but the Hello panel itself is not very
> keyboard-friendly itself.

I had to think about this for a bit.
I'd prefer to have some functionality mirrored in a submenu too. This would drive the accessibility of Hello forward in a neat way and make the feature more discoverable too.

We could start off with a single menu item in the Tools menu with one of the labels that Matej provided. Then file a follow-up for the submenu where we decide which items should be in there and how they should work.

Outstanding question for the single-menu-item case: what will happen when the Hello button is in the palette/ not available? Will the item be disabled? If so, how will we communicate that the button needs to move to the toolbar or menu-panel to enable it?

	Sevaan Franks [:sevaan]
	Comment 8 • 9 years ago

I think Command + Y/Windows Key + Y could be used to open the Hello panel.

This could open the panel and move focus to it. Then the user can keyboard down, first through the Conversations/Contacts tab headers, then through the conversations. Hitting enter opens one and moves focus to the conversation window. If they keep keyboarding down they can hit the "Start a Conversation" button which opens a new room and moves focus there. All this plus a tab order.

Thoughts?

	:Gijs (he/him) Assignee
	Comment 9 • 9 years ago

(In reply to Sevaan Franks [:sevaan] from comment #8)
> I think Command + Y/Windows Key + Y could be used to open the Hello panel.
> 
> This could open the panel and move focus to it. Then the user can keyboard
> down, first through the Conversations/Contacts tab headers, then through the
> conversations. Hitting enter opens one and moves focus to the conversation
> window. If they keep keyboarding down they can hit the "Start a
> Conversation" button which opens a new room and moves focus there. All this
> plus a tab order.
> 
> Thoughts?

Windows key shortcuts are normally reserved for the OS. cmd-y is "redo" on OS X.

	Sevaan Franks [:sevaan]
	Comment 10 • 9 years ago

Shoot, I knew that. I guess we'll have to switch to a Cmd-Shift type of deal.

	Marco Zehe (:MarcoZ) Reporter
	Comment 11 • 9 years ago

Hi Sevaan, since finding a free keyboard shortcut is a rather difficult task, to move this forward, what about adding it as a menu item to a prominent spot near the top of the Tools menu? If we want to and find a keyboard shortcut in the long run, we can always add it.

	Sevaan Franks [:sevaan]
	Comment 12 • 9 years ago

(In reply to Marco Zehe (:MarcoZ) from comment #11)
> Hi Sevaan, since finding a free keyboard shortcut is a rather difficult
> task, to move this forward, what about adding it as a menu item to a
> prominent spot near the top of the Tools menu? If we want to and find a
> keyboard shortcut in the long run, we can always add it.

Love it. Let's do that.

	(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo
	Comment 13 • 9 years ago

Removing my needinfo since Sevaan has already taken care of it :)

	:Gijs (he/him) Assignee
	Comment 15 • 9 years ago

Shell/Mike, what would it take to get this prioritized? Our a11y team is getting flak from users because they still can't use Hello, and just adding a menuitem to open the panel (probably hiding the menuitem if the button is not in the UI) should be relatively easy to do.

	Dan Mosedale (:dmosedale, :dmose)
	Comment 16 • 9 years ago

What would we want the menuitem to say?  Seems like "Firefox Hello..." would be the most straightforward, but it's not likely to be very meaningful to people who don't already know what it is.  "Join a Room..." seems like it would have a similar problem.  "Make a Call..." feels wrong.

Sevaan, two questions: does the mechanics of Gijs' suggestion in the previous comment work for you?  If we were to do a menu item, what would you suggest calling it?

	Matej Novak [:matej]
	Comment 17 • 9 years ago

I would go with something like "Open Firefox Hello…" or "Open Hello…"

"Join a room" seems even more confusing and we don't want to use the word "call" in relation to Hello.

	Sevaan Franks [:sevaan]
	Comment 18 • 9 years ago

(In reply to Dan Mosedale (:dmose) - needinfo? me for response from comment #16)
> Sevaan, two questions: does the mechanics of Gijs' suggestion in the
> previous comment work for you?  If we were to do a menu item, what would you
> suggest calling it?

Yes. But is opening the panel enough? A user needs to be able to actual open a conversation or start a new one. https://bugzilla.mozilla.org/show_bug.cgi?id=1106932#c8 has an approach that could handle that.

I like "Open Firefox Hello"

	:Gijs (he/him) Assignee
	Comment 19 • 9 years ago

(In reply to Sevaan Franks [:sevaan] from comment #18)
> (In reply to Dan Mosedale (:dmose) - needinfo? me for response from comment
> #16)
> > Sevaan, two questions: does the mechanics of Gijs' suggestion in the
> > previous comment work for you?  If we were to do a menu item, what would you
> > suggest calling it?
> 
> Yes. But is opening the panel enough? A user needs to be able to actual open
> a conversation or start a new one.
> https://bugzilla.mozilla.org/show_bug.cgi?id=1106932#c8 has an approach that
> could handle that.

For sure. I'm told this should already work (more or less), but it could be there is more work to do there. Opening the panel is kind of a baseline requirement though.
 
> I like "Open Firefox Hello"

Ditto. In terms of "it's not very meaningful to people who don't know what it is", I would say that the smilie-face-chat-bubble icon is not really much more meaningful (even if pictures are 1000 words rather than 3). ;-)

	Dan Mosedale (:dmosedale, :dmose)
	Comment 20 • 9 years ago

(In reply to :Gijs Kruitbosch from comment #19)
> >  
> > I like "Open Firefox Hello"
> 
> Ditto. In terms of "it's not very meaningful to people who don't know what
> it is", I would say that the smilie-face-chat-bubble icon is not really much
> more meaningful (even if pictures are 1000 words rather than 3). ;-)

Right now, the tooltip for the button says "Start a conversation".  Would that work as a menu item, if followed by an ellipsis?  It would both have more clarity, and keep the two pieces of UI speaking the same language...

	Matej Novak [:matej]
	Comment 21 • 9 years ago

(In reply to Dan Mosedale (:dmose) - needinfo? me for response from comment #20)
> (In reply to :Gijs Kruitbosch from comment #19)
> > >  
> > > I like "Open Firefox Hello"
> > 
> > Ditto. In terms of "it's not very meaningful to people who don't know what
> > it is", I would say that the smilie-face-chat-bubble icon is not really much
> > more meaningful (even if pictures are 1000 words rather than 3). ;-)
> 
> Right now, the tooltip for the button says "Start a conversation".  Would
> that work as a menu item, if followed by an ellipsis?  It would both have
> more clarity, and keep the two pieces of UI speaking the same language...

Yup, that would work, provided it isn't too long.

	Mark Banner (:standard8)
	Comment 22 • 9 years ago

Updating user story to reflect what's been decided and save searching through. I'm bumping the priority a bit, as I think its reasonable to have this in the next section of work that we do.

	:Gijs (he/him) Assignee
	Comment 23 • 9 years ago

Bug 1106932 - allow opening the loop panel from the menu, focus the panel immediately, r?mikedeboer

	:Gijs (he/him) Assignee
	Comment 24 • 9 years ago

This has a remaining issue that focus doesn't go into the panel the first time you open it, which seems to have to do with stuff not being ready yet or whatever. I spent half an hour trying to get that to work and gave up. Someone who knows loop needs to have a look at that.

Once this has landed, obviously work needs to start on actually making the rest of the thing keyboard-accessible. It used to be OK, from what I recall, but now once you start a conversation almost one of the buttons are keyboard-focusable. The conversation list buttons in the main panel which only appear when you hover over the item are also obvious candidates for improvement (making the list focusable and keyboard-navigable and showing the button on focused items in the list would be the sensible solution there).

	Marco Zehe (:MarcoZ) Reporter
	Comment 25 • 9 years ago

This is currently held up by bug 1210548, a hang when trying to focus the panel with F6 after it opens initially and doesn't get focus automatically. That happens even without this patch in a regular Nightly build.

	Mike de Boer [:mikedeboer]
	Comment 26 • 9 years ago

Comment on attachment 8668604 [details]
MozReview Request: Bug 1106932 - allow opening the loop panel from the menu, focus the panel immediately, r?mikedeboer

https://reviewboard.mozilla.org/r/21009/#review18969

Looks neat! Thanks.

::: browser/base/content/browser-loop.js:148
(Diff revision 1)
>              iframe.contentWindow.addEventListener("loopPanelInitialized", function loopPanelInitialized() {

Ah, yeah, this indenting business has been an eyesore for a while now - thanks for fixing!

	:Gijs (he/him) Assignee
	Comment 27 • 9 years ago

(In reply to Mike de Boer [:mikedeboer] from comment #26)
> Comment on attachment 8668604 [details]
> MozReview Request: Bug 1106932 - allow opening the loop panel from the menu,
> focus the panel immediately, r?mikedeboer
> 
> https://reviewboard.mozilla.org/r/21009/#review18969
> 
> Looks neat! Thanks.
> 
> ::: browser/base/content/browser-loop.js:148
> (Diff revision 1)
> >              iframe.contentWindow.addEventListener("loopPanelInitialized", function loopPanelInitialized() {
> 
> Ah, yeah, this indenting business has been an eyesore for a while now -
> thanks for fixing!

Landed but with a different accesskey (t) because I realized "s" conflicted with "Sync now".


remote:   https://treeherder.mozilla.org/#/jobs?repo=fx-team&revision=58acf8a0176e

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 28 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/58acf8a0176e

	Marco Zehe (:MarcoZ) Reporter
	Comment 29 • 9 years ago

Comment on attachment 8668604 [details]
MozReview Request: Bug 1106932 - allow opening the loop panel from the menu, focus the panel immediately, r?mikedeboer

Clearing feedback request since I gave the feedback verbally, and this has landed by now.