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 726265 - Allow interface viewer states to fill horizontally.
Allow interface viewer states to fill horizontally.
Status: RESOLVED OBSOLETE
Product: accerciser
Classification: Applications
Component: plugins
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Accerciser Developers
Accerciser Developers
Depends on:
Blocks:
 
 
Reported: 2014-03-13 18:18 UTC by Joseph Scheuhammer
Modified: 2021-05-17 15:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Shows the bad display of the states requiring horizontal scrolling. (44.06 KB, image/png)
2014-03-13 18:18 UTC, Joseph Scheuhammer
  Details
One line patch to allow states view to expand horizontally. (1.08 KB, patch)
2014-03-13 18:21 UTC, Joseph Scheuhammer
reviewed Details | Review
Uses a paned for the states - attributes views (25.10 KB, patch)
2014-03-18 18:18 UTC, Joseph Scheuhammer
committed Details | Review

Description Joseph Scheuhammer 2014-03-13 18:18:08 UTC
Created attachment 271777 [details]
Shows the bad display of the states requiring horizontal scrolling.

The display of the list of states requires horizontal scrolling -- see the screen shot attachment.

This should have been fixed given the resolution of bugzilla 704550 and the associated commit:
https://git.gnome.org/browse/accerciser/commit/?id=1659fc5b47aa54bf25d6a8d49accdd09ba4b2e4b

I don't know why that no longer holds, but a forthcoming patch solves the problem.
Comment 1 Joseph Scheuhammer 2014-03-13 18:21:56 UTC
Created attachment 271778 [details] [review]
One line patch to allow states view to expand horizontally.
Comment 2 Javier Hernández 2014-03-13 22:57:20 UTC
Review of attachment 271778 [details] [review]:

Joseph,

thanks for reporting the issue and providing a patch for it.
However ... what do you think about putting those two widgets in a paned?
By using it, we can let the user to change its size easily.
Comment 3 Joseph Scheuhammer 2014-03-18 18:18:49 UTC
Created attachment 272318 [details] [review]
Uses a paned for the states - attributes views

(In reply to comment #2)
> Review of attachment 271778 [details] [review]:
> 
> Joseph,
> 
> thanks for reporting the issue and providing a patch for it.
> However ... what do you think about putting those two widgets in a paned?
> By using it, we can let the user to change its size easily.

At first I thought that a paned is overkill.  But after modifying the UI to use one, it's actually okay.

Find a patch that does both what the previous patch did, and wraps the states and attributes in a split pane (paned).

Note that I did this using glade, and a diff shows a lot of irrelevant differences.
Comment 4 Javier Hernández 2016-02-24 02:57:15 UTC
Review of attachment 272318 [details] [review]:

Sorry for the late response, but better late than never :/

Merged in https://git.gnome.org/browse/accerciser/commit/?id=01850e140fa96e77839f5399586070a87b9a8c47

Cheers!
Comment 5 GNOME Infrastructure Team 2021-05-17 15:44:36 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/accerciser/-/issues/40.