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 124157 - gfig - Lock Grid needs a tooltip
gfig - Lock Grid needs a tooltip
Status: VERIFIED FIXED
Product: GIMP
Classification: Other
Component: Plugins
1.x
Other All
: Low enhancement
: 2.0
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2003-10-08 21:24 UTC by Alan Horkan
Modified: 2003-10-17 19:09 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement



Description Alan Horkan 2003-10-08 21:24:20 UTC
The following items in Gfig dont have tooltips:
Snap to Grid, Display Grid and Lock Grid 

The first two are pretty much self explanatory but I'm not entirely sure
what Lock Grid actually does.  I'll look at the code of course but if it is
not obvious from just the label then a Tooltip would definately be a good
idea.  

I should note that Gfig is full of really useful tooltips which have made
it much easier to write the documetation I am currently working on (but
cant quite seem to finish).
Comment 1 Maurits Rijk 2003-10-09 06:58:09 UTC
Looking at the sourcecode reveals that Lock Grid actually does...
uhhh.. nothing ;) I don't have a clue what it is supposed to do, but
it is unimplemented yet. I propose to just remove this feature.

Alan, do you think we should still have tooltips for Snap to Grid and
Display Grid? If yes, please make a proposal, and I will add them to gfig.
Comment 2 Maurits Rijk 2003-10-09 07:45:39 UTC
Trivial to fix, therefor setting milestone to 2.0
Comment 3 Alan Horkan 2003-10-10 18:31:52 UTC
I mailed Andy Tai (original author of Gfig) about this and hopefully
I'll get a reply soon.  

If you do remove this from the user interface it might be best to just
comment it out and leave it in the code.  

As for Snap to Grid, Display Grid and Lock Grid I think they should
have tooltips but I'm not too sure what would be appropriate Tooltips
would be as they are pretty straight forward (although I am tempted to
change Display Grid to "Show Grid" and to put it before Snap to Grid
because I think it is logical towant to show the Grid before you start
snapping to it).  
Comment 4 Maurits Rijk 2003-10-17 11:46:24 UTC
2003-10-17  Maurits Rijk  <lpeek.mrijk@consunet.nl>

* plug-ins/gfig/gfig.c: commented out code for Lock Grid (which wasn't 
  implemented). Fixes #124157.

I also added tooltips, but they don't contain any useful information
yet. Alan, do you have a suggestion for the tips? Please e-mail
directly to me. I'm closing this bug.
Comment 5 Sven Neumann 2003-10-17 11:54:29 UTC
Maurits, I saw you also changed capitalization in the GFig plug-in.
Although the HIG suggests this style, I don't think it's a good idea
to start changing this now. It seems that consistency is more
important than HIG conformance and it is IMO too late to do this
change all over the place. After all you want to give the translators
a chance to finish there translations. Since we are close to a
release, string changes should be done with care. The capitalization
changes should be done after 2.0 is out.
Comment 6 Sven Neumann 2003-10-17 12:18:43 UTC
2003-10-17  Sven Neumann  <sven@gimp.org>
 
    * plug-ins/gfig/gfig.c: reverted most of Maurits changes (not the
    bug-fix though). The tentative feature freeze we are in also
    implies a string freeze.  It's too late for such massive string
    changes.
 
Comment 7 Raphaël Quinet 2003-10-17 12:39:06 UTC
I thought that the strings would only be frozen after the release of
2.0-rc1.  But maybe it makes sense to freeze them now.  Anyway, to be
sure that we do not forget to re-apply Maurits' patch, I have now
opened a separate bug report: see bug #124834.
Comment 8 Sven Neumann 2003-10-17 13:17:05 UTC
I closed that report as a duplicate of bug #123699.

There is no strict string freeze in effect yet but it's definitely too
late to change the capitalization since that would affect several
hundreds if not thousands of strings. This is not an option since we
target a prerelease in a few weeks and we want to ship 2.0 with
complete translations.
Comment 9 Alan Horkan 2003-10-17 19:09:46 UTC
Andy said he just never quite got around to implementing lock Grid.  

I'm satisfied with Maurits patch, thanks.