Popup window for "Share this page" button shouldn't be autohide (or should restore inputted text when the panel is reopened for same tab)

RESOLVED WORKSFORME

Status

()

Firefox
SocialAPI
RESOLVED WORKSFORME
a year ago
3 months ago

People

(Reporter: masayuki, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox51 affected)

Details

The popup window for "Share this page" button is specified as autohide in current build. However, it's not useful and there are no reasonable reasons.

First of all, the reason why most popup XUL <panel> should be autohide is that if it's a top-most window, there may not be a way to close it. In such case, top-most window in OS level may block other applications' UI. However, we can provide a close button for it, and on non-Linux platforms, the popup is NOT a top-most window. So, if we add a close button to the popup, we don't worry about this security issue.

Then, I'm in trouble in some pages, when I want to share a page and I want to paste two or more separated text in the page to the message, I cannot do it because when I move focus to the page for selecting text, the popup is closed by autohide behavior.

So, I'd be happy if the popup isn't autohide and has close button for Linux.
https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XUL/Attribute/panel.noautohide
OS: Unspecified → All
Hardware: Unspecified → All

Comment 1

a year ago
Shane, thoughts?

(In reply to Masayuki Nakano [:masayuki] (Mozilla Japan) from comment #0)
> The popup window for "Share this page" button is specified as autohide in
> current build. However, it's not useful and there are no reasonable reasons.
> 
> First of all, the reason why most popup XUL <panel> should be autohide is
> that if it's a top-most window, there may not be a way to close it. In such
> case, top-most window in OS level may block other applications' UI. However,
> we can provide a close button for it, and on non-Linux platforms, the popup
> is NOT a top-most window. So, if we add a close button to the popup, we
> don't worry about this security issue.

Well, we'd also need to hide it if another window or application is focused (certainly on Linux, because the panel will be always-on-top), if the user switches tabs, ... there are a lot of edgecases to consider if it isn't "autohide", and you're basically suggesting that they are less important than your usecase. I think that is not necessarily true.
Component: Disability Access → SocialAPI
Flags: needinfo?(mixedpuppy)
I don't see where noautohide is being set at all.

https://dxr.mozilla.org/mozilla-central/source/browser/base/content/browser.xul#253
Flags: needinfo?(mixedpuppy)
ie. afaik the share panel has default behavior.

Comment 4

a year ago
(In reply to Shane Caraveo (:mixedpuppy) from comment #3)
> ie. afaik the share panel has default behavior.

Yes, the request is for it to *not* be autohidden by default, ie to *add* the "noautohide" flag.
But wouldn't that make the share panel behave different than other panels?
ie. I don't get it.  If default behavior is a problem with the share panel, it is a problem with all panels and the default should change.
(In reply to :Gijs Kruitbosch (PTO recovery mode) from comment #1)
> Shane, thoughts?
> 
> (In reply to Masayuki Nakano [:masayuki] (Mozilla Japan) from comment #0)
> > The popup window for "Share this page" button is specified as autohide in
> > current build. However, it's not useful and there are no reasonable reasons.
> > 
> > First of all, the reason why most popup XUL <panel> should be autohide is
> > that if it's a top-most window, there may not be a way to close it. In such
> > case, top-most window in OS level may block other applications' UI. However,
> > we can provide a close button for it, and on non-Linux platforms, the popup
> > is NOT a top-most window. So, if we add a close button to the popup, we
> > don't worry about this security issue.
> 
> Well, we'd also need to hide it if another window or application is focused
> (certainly on Linux, because the panel will be always-on-top), if the user
> switches tabs, ... there are a lot of edgecases to consider if it isn't
> "autohide", and you're basically suggesting that they are less important
> than your usecase. I think that is not necessarily true.

Yes, that's difficult point.

(In reply to Shane Caraveo (:mixedpuppy) from comment #6)
> ie. I don't get it.  If default behavior is a problem with the share panel,
> it is a problem with all panels and the default should change.

The reason why current panel for "Share this page" is not enough to work well like the other panel is, users may input too complicated text on auto-hide panel.

In other words, when it's hidden automatically, user may want to restart to input. If the button stores inputted text until the tab is closed (or until active tab is switched) and the text is restored when the panel is reopened, it's enough to me.
Summary: Popup window for "Share this page" button shouldn't be autohide → Popup window for "Share this page" button shouldn't be autohide (or should restore inputted text when the panel is reopened for same tab)

Comment 8

3 months ago
We removed the share panel.
Status: NEW → RESOLVED
Last Resolved: 3 months ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.