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 778843 - bookmarks dialog resizable, contents fixed size
bookmarks dialog resizable, contents fixed size
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Bookmarks
3.23.x
Other Linux
: Normal minor
: ---
Assigned To: Jonathan Kang
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-02-17 16:38 UTC by Marinus Schraal
Modified: 2017-12-19 02:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
bookmarks dialog (25.80 KB, image/png)
2017-02-17 16:38 UTC, Marinus Schraal
  Details
bookmarks: Don't set scrolled window's max-content-height (1.28 KB, patch)
2017-12-06 10:05 UTC, Jonathan Kang
committed Details | Review
fix 4a32a783195b85a819e3ef6530fc5e2b4ba7b049 (1.22 KB, patch)
2017-12-18 09:15 UTC, Jonathan Kang
committed Details | Review

Description Marinus Schraal 2017-02-17 16:38:10 UTC
Created attachment 346086 [details]
bookmarks dialog

The bookmarks dialog is resizeable (made larger), but the contents are fixed vertically.

It would make sense to allow for the tags section to be bigger and show more tags. Right now it scales horizontally, however it only scales the size of the tag and doesn't show more of them.
Comment 1 Michael Catanzaro 2017-02-19 19:50:46 UTC
I'm not likely to have time to look into this soon, unfortunately, and I really don't understand how GTK+ sizing works. :(  If anyone wants to submit a patch, it would be much appreciated.
Comment 2 Jonathan Kang 2017-12-06 10:05:16 UTC
Created attachment 365094 [details] [review]
bookmarks: Don't set scrolled window's max-content-height
Comment 3 Michael Catanzaro 2017-12-06 12:58:08 UTC
Review of attachment 365094 [details] [review]:

OK, thanks!
Comment 4 Michael Catanzaro 2017-12-06 12:58:48 UTC
This doesn't change the size of the dialog itself, right? Only the scrolled window within it?
Comment 5 Jonathan Kang 2017-12-08 02:14:53 UTC
(In reply to Michael Catanzaro from comment #4)
> This doesn't change the size of the dialog itself, right? Only the scrolled
> window within it?

Yes and yes. The size of the scrolled window will change when users resize the
whole dialog.
Comment 6 Michael Catanzaro 2017-12-08 16:04:15 UTC
OK, great. Feel free to commit it.
Comment 7 Jonathan Kang 2017-12-11 09:55:43 UTC
Comment on attachment 365094 [details] [review]
bookmarks: Don't set scrolled window's max-content-height

Pushed to master as commit 4a32a783195b85a819e3ef6530fc5e2b4ba7b049.
Comment 8 Jonathan Kang 2017-12-18 09:15:28 UTC
Created attachment 365674 [details] [review]
fix 4a32a783195b85a819e3ef6530fc5e2b4ba7b049

Sorry, the previous patch doesn't work very well. This patch fixes that.

Tested and works at my side.
Comment 9 Michael Catanzaro 2017-12-18 14:34:45 UTC
Review of attachment 365674 [details] [review]:

OK. Maybe avoid using a hash in the first line of the commit message, though.
Comment 10 Jonathan Kang 2017-12-19 02:34:54 UTC
Comment on attachment 365674 [details] [review]
fix 4a32a783195b85a819e3ef6530fc5e2b4ba7b049

Modified the commit message and pushed to master as
commit bdc9ed9604b7fd1ff238a7d1a0696b207165791c.