866413 - [meta] Electrolysis: Make Page Information dialog work

Status: RESOLVED FIXED

(Firefox :: Page Info Window, defect)

Comment 1

[:Gavin Sharp [email: [e-mail]]]

We've been considering removing this dialog for a while now, that may be the easiest way forward.

Comment 2

[Tom S [:evilpie]]

It actually kind of works with the the CPOW, but of course this is not the ideal solution. It seems to some kind of frame or image iteration with a callback, this is obviously nearly impossible to support without a content script.

Comment 3

[Tanner M. Young [:tmyoung]]

The dialog is still useful--and useful as a pop-out window as it has information about the window the user is examining (but it could be made into a sidebar or something similar like Firebug does).  Three years ago there were plans to extensively rework Page Info to be a window like about:addons and similar pages.  However, there wasn't enough UX, developer, peer, and QA consensus to see it through.  I've wanted to see (and help) Page Info get a face lift and be better integrated with the Firefox of 2013 than the Firefox of bygone days.  However, I wouldn't want to the see dialog entirely removed.  Is there an active newsgroup discussion on this?  (I can't find one, and it's probably better to discuss the wider philosophical questions about Page Info in there instead of bogging down the bug with many comments.)

I'd also be interested in the opinions of the Seamonkey developers as they keep an actively developed Page Info dialog for their suite that often imports from Firefox, so they probably have some useful insights.

Comment 4

[Caspy7]

When on a page, Page Info is the easiest way to affect [the growing list of] site permissions.

Removing Page Info would be a pretty huge UX change and, I agree, requires a wider discussion.

Comment 5

[:Gavin Sharp [email: [e-mail]]]

We're not going to remove page info as a side effect of a comment I make on an unrelated bug, you don't need to worry about that. This bug really isn't the best place to get into the pros/cons.

Comment 6

[Chris Peterson [:cpeterson]]

Mass tracking-e10s flag change. Filter bugmail on "2be0fcce-e36a-4e2c-aa80-0e3d33eb5406".

Comment 7

[:Gijs (out for now; he/him)]

Please don't forget to re-enable the tests for this feature if/when this gets fixed.

Comment 8

[Elbart]

Is the bug of "Use Default" not activating the options next to it part of this bug, or would it require a new bug to be filed?

Comment 9

[:Gijs (out for now; he/him)]

(In reply to Elbart from comment #8)
> Is the bug of "Use Default" not activating the options next to it part of
> this bug, or would it require a new bug to be filed?

I don't know what you mean by "Use Default" as you gave almost no context - do you mean the permissions part of the page info window, or something else?

AFAIK, everything inside the page info window that doesn't currently work with e10s and works outside e10s is covered by this bug. Everything else is worth filing a new bug for (after searching for dupes etc.)

Comment 11

[Jared Wein [:jaws] (please needinfo? me)]

Bug 1112292 was tracking-e10s: m7+. Can that tracking flag be brought over here?

Comment 12

[Mike Conley (:mconley) (:⚙️)]

What about the Page Info dialog is not working? I see that it's slow and janky, but that's going to be fixed in bug 1040947.

I know the Feeds bit is busted - that's bug 1109570.

Besides those two things, what functionality is currently busted in the Page Info dialog?

Comment 13

[Mike Conley (:mconley) (:⚙️)]

Or is this a metabug for the dependencies?

Comment 14

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

Bug 1159616.

Comment 15

[Tom S [:evilpie]]

Page Info still feels really slow sometimes. Besides that when I filled this CPOWs didn't support some of the required features, i.e. they weren't bi-directional.

Comment 16

[Mike Conley (:mconley) (:⚙️)]

Ok. In that case, I think this is becoming a metabug to deal with specific Page Info issues.

We don't track meta bugs - instead, I've made sure the dependencies are being tracked.

Comment 18

[Henrik Skupin [:whimboo][⌚️UTC+2]]

This is a meta bug, so bug 1040947 should be added as dependency.

Comment 19

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:florian, could you have a look please?
For more information, please visit auto_nag documentation.