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 315443 - dropdown widgets should remember the view type (grid/list)
dropdown widgets should remember the view type (grid/list)
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: User Interface
git master
Other All
: Normal minor
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2005-09-07 12:26 UTC by Jakub Steiner
Modified: 2006-01-14 00:10 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Jakub Steiner 2005-09-07 12:26:51 UTC
The dropdown widgets in the tool settings dock should remember the view type the
user specified last as for some data  and types of users a grid view makes more
sense than others. 

Based on my work patterns I also suggest using grid view by default for brushes
and patterns while keeping list view for fonts.

Other information:
Comment 1 Tuomas Kuosmanen 2005-09-07 12:32:24 UTC
I agree, the setting should be "sticky" - there is no point in having a
selection there without remembering it.

I also agree for brushes the grid is better (since there's just the rectangular
preview" - and for fonts list works better since you also want to see the font
name. By the way, would it be technically unfeasible to actually render the font
name in the font itself to the list? I know it'd suck for symbol-like fonts -
but the list could have a little area for the font name below where it would be
shown in the widget font.
Comment 2 Sven Neumann 2005-09-07 15:03:13 UTC
Tuomas, please don't mix different issues in bug reports. There is already a
number of bug report dealing with enhancements to the fonts dialog, bug #137624
probably comes closest.
Comment 3 Michael Natterer 2005-09-10 20:29:19 UTC
Quite easy to implement. Setting milestone to 2.4
Comment 4 Michael Natterer 2005-09-10 20:45:12 UTC
Hm, it should propably remember the preview size too?
Comment 5 Michael Natterer 2005-10-31 02:13:09 UTC
A bit better now...

2005-10-31  Michael Natterer  <mitch@gimp.org>

	* app/widgets/gimpviewablebutton.[ch] (gimp_viewable_button_new):
	add GimpViewType parameter.

	* app/widgets/gimptemplateeditor.c
	* app/widgets/gimpviewablebox.c: pass view types, using grid view
	for brushes and patterns ass suggested in bug #315443.
Comment 6 Michael Natterer 2006-01-14 00:10:25 UTC
Fixed in CVS:

2006-01-14  Michael Natterer  <mitch@gimp.org>

	* app/config/gimpconfig-utils.[ch] (gimp_config_connect_full):
	variant of gimp_config_connect() which allows the connected
	objects to have different propertynames.

	* app/widgets/widgets-enums.[ch]: removed enum GimpViewType...

	* app/core/core-enums.[ch]: ...and added it here.

	* app/widgets/gimpviewablebutton.[ch] (gimp_viewable_button_new):
	added "button_preview_size" parameter so the button and popup
	preview sizes can be specified separately.

	* app/widgets/gimptemplateeditor.c: changed accordingly.

	* app/widgets/gimpviewablebox.[ch] (gimp_prop_*_box_new):
	new functions which take additional "view_type_prop" and
	"view_size_prop" parameters and sync the passed context's
	properties with the resp. properties of the viewable button.

	* app/paint/gimppaintoptions.[ch]
	* app/tools/gimpbucketfilloptions.c
	* app/tools/gimpclonetool.c
	* app/tools/gimppaintoptions-gui.c
	* app/tools/gimptextoptions.[ch]: added view-type and view-size
	properties to the options objects and use the new viewable box
	constructors so the selected view types and sizes are persistant
	across sessions. Fixes bug #315443.