After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 723238 - popup windows should respect the window properties
popup windows should respect the window properties
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Interface
git master
Other Mac OS
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
: 720601 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-01-29 14:26 UTC by Carlos Garcia Campos
Modified: 2014-02-05 17:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Remove chrome and popup parameters from ephy_shell_new_tab_full (6.51 KB, patch)
2014-01-29 14:29 UTC, Carlos Garcia Campos
committed Details | Review
Move the window geometry handling from EphyEmbed to EphyWindow (3.38 KB, patch)
2014-01-29 14:31 UTC, Carlos Garcia Campos
committed Details | Review
Remove show-toolbars setting (5.23 KB, patch)
2014-01-29 14:32 UTC, Carlos Garcia Campos
committed Details | Review
Make sure the toolbar is always visible (3.19 KB, patch)
2014-01-29 14:32 UTC, Carlos Garcia Campos
committed Details | Review
Move the UI chrome handling to EphyWindow (15.19 KB, patch)
2014-01-29 14:33 UTC, Carlos Garcia Campos
committed Details | Review
popup windows should respect the window properties (7.03 KB, patch)
2014-01-29 14:33 UTC, Carlos Garcia Campos
committed Details | Review
Make the location entry non-editable for popups (964 bytes, patch)
2014-01-29 14:34 UTC, Carlos Garcia Campos
committed Details | Review

Description Carlos Garcia Campos 2014-01-29 14:26:52 UTC
When a popup window is created (window.open()) the window features (WebKitWindowProperties) should be honored.
Comment 1 Carlos Garcia Campos 2014-01-29 14:29:51 UTC
Created attachment 267525 [details] [review]
Remove chrome and popup parameters from ephy_shell_new_tab_full
Comment 2 Carlos Garcia Campos 2014-01-29 14:31:40 UTC
Created attachment 267526 [details] [review]
Move the window geometry handling from EphyEmbed to EphyWindow
Comment 3 Carlos Garcia Campos 2014-01-29 14:32:09 UTC
Created attachment 267527 [details] [review]
Remove show-toolbars setting
Comment 4 Carlos Garcia Campos 2014-01-29 14:32:36 UTC
Created attachment 267528 [details] [review]
Make sure the toolbar is always visible
Comment 5 Carlos Garcia Campos 2014-01-29 14:33:04 UTC
Created attachment 267529 [details] [review]
Move the UI chrome handling to EphyWindow
Comment 6 Carlos Garcia Campos 2014-01-29 14:33:40 UTC
Created attachment 267530 [details] [review]
popup windows should respect the window properties
Comment 7 Carlos Garcia Campos 2014-01-29 14:34:09 UTC
Created attachment 267531 [details] [review]
Make the location entry non-editable for popups
Comment 8 Claudio Saavedra 2014-01-29 14:38:57 UTC
Review of attachment 267525 [details] [review]:

Makes sense.
Comment 9 Carlos Garcia Campos 2014-01-30 07:09:32 UTC
*** Bug 720601 has been marked as a duplicate of this bug. ***
Comment 10 Claudio Saavedra 2014-02-04 12:41:14 UTC
Review of attachment 267526 [details] [review]:

OK
Comment 11 Claudio Saavedra 2014-02-04 13:07:44 UTC
Review of attachment 267527 [details] [review]:

Fine by me.
Comment 12 Claudio Saavedra 2014-02-04 13:10:00 UTC
Review of attachment 267528 [details] [review]:

You mean always active, right?
Comment 13 Carlos Garcia Campos 2014-02-04 13:42:35 UTC
(In reply to comment #12)
> Review of attachment 267528 [details] [review]:
> 
> You mean always active, right?

I mean always visible, with the toolbar hidden there's no chrome in the window, so it's not obvious how to close the window, and I don't think it's even possible to move it.
Comment 14 Claudio Saavedra 2014-02-04 15:44:14 UTC
Comment on attachment 267528 [details] [review]
Make sure the toolbar is always visible

I see. Just wondering whether the patch should be merged with the previous one, but I think it looks fine.
Comment 15 Claudio Saavedra 2014-02-05 16:52:07 UTC
Review of attachment 267529 [details] [review]:

I guess this is good.
Comment 16 Claudio Saavedra 2014-02-05 16:52:47 UTC
Review of attachment 267530 [details] [review]:

OK
Comment 17 Claudio Saavedra 2014-02-05 16:54:52 UTC
Review of attachment 267531 [details] [review]:

OK