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 311353 - gswitchit: please use long description instead of short description
gswitchit: please use long description instead of short description
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: keyboard indicator (gswitchit)
2.10.x
Other Linux
: Normal minor
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-07-23 18:03 UTC by Loïc Minier
Modified: 2005-07-24 08:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
normal description instead of short description (631 bytes, patch)
2005-07-23 18:04 UTC, Loïc Minier
none Details | Review

Description Loïc Minier 2005-07-23 18:03:39 UTC
Hi,

Matijs van Zuijlen reported in Debian bug http://bugs.debian.org/319605 that
some schemes are undistinguishable via their shortDescription.

I propose using the normal description, and will attach the trivial patch.

Here's what the submitter proposed:
> I currently see the following possible solutions:
>.
> - Make xfree86.xml (or rather, xorg.xml) contain unique short descriptions
>   for every keyboard layout and variant (Doesn't seem likely, since at
>   least the English short descriptions seem to standardize on three-letter
>   abbreviations).
> - Add some disambiguation in the indicator if necessary given the user's
>   configuration. These could be either numbers (not so nice), or the name
>   field, like so: USA (us); USA (en_US); USA (us_intl), or, when using the
>   intl variant of the us layout (I don't know why there are two of them):
>   USA (intl).
> - Allow the user to set the display string (This would be rather awkward, I
>   suppose).

Bye,
Comment 1 Loïc Minier 2005-07-23 18:04:33 UTC
Created attachment 49632 [details] [review]
normal description instead of short description

(Makes the applet a bit long in the panel though.)
Comment 2 Sergey V. Udaltsov 2005-07-23 19:13:44 UTC
Loic, you are absolutely right saying 'a bit long in the panel'. Actually, I
introduced the shortDescription field into the xml format just in order to make
the applet usable. With the long description, it would take TOO much space. I
don't want it. Regarding the uniqueness of the shortDescriptions - it can be
discussed but I'd like to put some restrictions here:
1. User should not see layout/variant IDs. It is internal technical information
(like primary keys in the database)
2. The IDs should be short (no longer than 5 symbols, I'd say).
Sure, if you still think that longDescription is a WTG - you can raise the
question in desktop-devel-list.
Comment 3 Loïc Minier 2005-07-23 19:46:22 UTC
What do you think of:
> - Add some disambiguation in the indicator if necessary given the user's
>   configuration. These could be either numbers (not so nice), or the name
>   field, like so: USA (us); USA (en_US); USA (us_intl), or, when using the
>   intl variant of the us layout (I don't know why there are two of them):
>   USA (intl).

As I understand it, this would mean use short descriptions when no confusion if
possible, and add something discriminatory when necessary, such as the name, or
maybe switch to the normal description when two short descriptions are identical.

Would that be acceptable?
Comment 4 Sergey V. Udaltsov 2005-07-23 20:16:08 UTC
OK, let's consider two scenarious. First, I am using Russian and US layout. No
confision, just 'USA' and 'Rus' - short descriptions work ok.
Next, let's talk about some Canadian who would use two variants (Multilingual,
first and second). You are right, now he should feel uncomfortable seeing 'Can'
all the way. What would be the solution allowing to distinguish two variants?

Can(1) / Can(2)?

Can(XX) / Can(YY) - what would you see as XX and YY? max(strlen(XX)) = 3

Something else?
Comment 5 Loïc Minier 2005-07-23 20:28:38 UTC
For strlen(XX) <= 3, I can only think of numbers.  :-/

Since the description is in the right click menu, I think that's completely
acceptable.

Comment 6 Sergey V. Udaltsov 2005-07-23 20:42:25 UTC
OK, I generally agree. So if you have time right now to implement it - you are
very welcome. Unless I find time to do it in a next week or so.
Comment 7 Loïc Minier 2005-07-23 20:56:47 UTC
(Reopening.)
Comment 8 Sergey V. Udaltsov 2005-07-24 00:11:39 UTC
I made it even shorter. Right now, I see on my screen the sequence:

USA
Can
Can*
Can**

At least people would be able to distinguish. And it is still compact. Objections?
Comment 9 Loïc Minier 2005-07-24 08:52:13 UTC
Looks fine to me, you rock!