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 313674 - Width of image information side pane
Width of image information side pane
Status: RESOLVED FIXED
Product: eog
Classification: Core
Component: image viewer
git master
Other Linux
: Normal normal
: ---
Assigned To: EOG Maintainers
EOG Maintainers
: 317678 (view as bug list)
Depends on: 315189
Blocks:
 
 
Reported: 2005-08-16 22:40 UTC by Martin Ejdestig
Modified: 2005-10-28 02:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot showing the side pane occyping more than 1/3 of the window. (232.58 KB, image/png)
2005-08-16 22:41 UTC, Martin Ejdestig
  Details
Suggested patch (2.10 KB, patch)
2005-10-27 03:29 UTC, Ed Catmur
rejected Details | Review
Revised patch (1.91 KB, patch)
2005-10-27 19:47 UTC, Ed Catmur
none Details | Review

Description Martin Ejdestig 2005-08-16 22:40:22 UTC
The default width of the image information pane makes it occupy more than 1/3 of
the window meaning less space for the image and much empty space in the side
pane (at least when only the File tab is available). Since the primary thing you
want to do with EOG is view images... well, this is no good. :)

I understand that one want's all information in the side pane displayed without
the need of a horizontal scroll bar but it should be possible to measure the
width needed and set the width of the pane to only that.
Comment 1 Martin Ejdestig 2005-08-16 22:41:47 UTC
Created attachment 50819 [details]
Screenshot showing the side pane occyping more than 1/3 of the window.
Comment 2 Martin Ejdestig 2005-08-16 22:47:54 UTC
If measuring the space needed is a complicated exercise, perhaps setting it to
something smaller by default (but still enough for most cases... 1/4?) and
remembering where the user puts it is enough?
Comment 3 Allison Karlitskaya (desrt) 2005-09-05 14:57:55 UTC
  Thanks for the report.  Bug #315189 was just Fixed in CVS HEAD, thus fixing
your problems.
Comment 4 Martin Ejdestig 2005-09-05 19:16:20 UTC
Yes, it fixes the problem "takes up 1/3 of the width" but now the width of the
information pane is to small. Only the "Attribute" column is visible.

Otherwise it works correctly. At least that's what I think. That is, there's no
empty space in the "image area" because the image has been made smaller to make
the information pane fit in the window.

(Btw, the patch hasn't hit anoncvs yet so I hope I didn't miss anything by
making the one line modification myself...)
Comment 5 Sebastien Bacher 2005-10-01 12:01:59 UTC
*** Bug 317678 has been marked as a duplicate of this bug. ***
Comment 6 Ed Catmur 2005-10-27 03:29:01 UTC
Created attachment 53933 [details] [review]
Suggested patch

Patch against eog 2.12.1.

Make image information side pane non-scrolled, and make the side pane not
shrinkable smaller than its minimum allocation.

Also, set the initial geometry after the image list is set up. (This makes
geometry calculation more accurate).

Result is that when eog starts the image is (if possible) displayed at 100%
with the image information side pane at minimum allocation (column headers
fully visible) and no gaps anywhere.
Comment 7 Ed Catmur 2005-10-27 03:54:41 UTC
Forget that, it doesn't work (fails with very long file names).

Seeing as the widths of the two sides of the vpane can take any value, perhaps
just set the size request on info_view to some arbitrary width (the width of
some arbitrary string)?
Comment 8 Lucas Rocha 2005-10-27 03:59:53 UTC
The only thing you need to fix here is the initial side pane width that must
have column headers fully visible. Then, the user can resize it as (s)he wants.
Comment 9 Ed Catmur 2005-10-27 19:47:50 UTC
Created attachment 53962 [details] [review]
Revised patch

This patch sets the size request of the info view pane to 3/2 the width of the
requisition of the file info treeview (when the file info treeview is empty, so
the file name does not affect this). I set it to 3/2 the width because the
width requisiton of the file info treeview is enough to show the column headers
(obviously!) but chops off most file names.

Also, in eog_info_view_file_show_data() (and the same for the exif view) I set
an idle handler to autosize the columns. It appears if this is not done the
columns will not be autosized, so the scrollbar fails to appear even if needed.
Comment 10 Lucas Rocha 2005-10-28 02:50:46 UTC
The patch is OK though the autosize calls are not necessary here. The scrollbar
works nice when needed. Applied without the idle calls, thanks!