Bug 194320 - Review Request: im-chooser
Summary: Review Request: im-chooser
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Cantrell
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 182541 FC-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-06-07 09:08 UTC by Akira TAGOH
Modified: 2013-01-10 01:25 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-20 20:24:13 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Akira TAGOH 2006-06-07 09:08:16 UTC
Spec URL: http://people.redhat.com/tagoh/im-chooser/im-chooser.spec
SRPM URL: http://people.redhat.com/tagoh/im-chooser/im-chooser-0.1.0-1.src.rpm
Description:
im-chooser is a GUI configuration tool for Input Method that easily choose what you would use for your desktop.

Comment 1 Akira TAGOH 2006-06-07 09:27:23 UTC
SRPM URL: http://people.redhat.com/tagoh/im-chooser/im-chooser-0.1.1-1.src.rpm

Please use this one instead - actually no changes in spec file, but in upstream
tarball.

Comment 2 Bill Nottingham 2006-06-07 14:37:32 UTC
Don't we only ship SCIM?

Comment 3 Akira TAGOH 2006-06-08 00:10:22 UTC
Yes, but this is still helpful for people who don't want to use IM and we are
trying to enable IM by default. ideally adding IM configuration page into
firstboot  too would be great, though.

Comment 4 Bill Nottingham 2006-06-08 03:04:28 UTC
I don't get it.

If we only have one IM, what's to configure?

If it's so obtrusive that it would need to be turned off (aside from just not
installing it), it needs to be fixed.

Comment 5 Akira TAGOH 2006-06-08 06:34:27 UTC
Ok, that makes sense. I'll put this into Extras then.

Comment 6 Leon Ho 2006-07-05 07:05:10 UTC
Hi Bill, let me try to justify this before creating a request solely to RHEL
release. There are user preferences where user would like to install language
packs in anaconda for reading web pages, but not input the native scripts.
Especially for now even en_US users will load SCIM if they have installed SCIM
packages. There should be a quick way for user in GUI to disable/enable IM
temporary - having the user to disable SCIM by uninstalling them by rpm command
is probably low in usability.

We consider this tool just like an config tool for user to config their keyboard
(enable/disable their native key maps), or enabling/disabling samba service. All
those actions do not require uninstalling the packages should they want to
disable the function, instead we have a good user interface for them to disable
temporary. In this sense, we should have similar GUI tool for Input Method.

In the long term wise, we want to standardize this IM configuration and merged
it into Keyboard properties.



Comment 7 Bill Nottingham 2006-07-05 17:59:31 UTC
How is it obtrusive if it's installed but not generally used - where are users
going to bump into it? 

Comment 8 Leon Ho 2006-07-06 01:24:49 UTC
It will be mainly on the trayicon on the panel, and also the trigger key for
toggle the input method. As those lead to more a preference of usage so it is
solely providing a preference for user to enable/disable the service without
uninstall the package from the command line.

Comment 9 Bill Nottingham 2006-07-06 01:36:33 UTC
So, it's .... removing something from the autostart folder on the desktop?
Something else?

Comment 10 Warren Togami 2006-07-06 01:54:44 UTC
Note that this is comparable to how you enable IM on Windows or Mac.  If you are
logged into an English desktop, you find IM options in desktop preferences
somewhere and enable it.  Then from then on IM is visible from your systray and
active within desktop applications.

We definitely need something comparable to this in our base distribution.

Comment 11 Leon Ho 2006-07-06 02:54:24 UTC
notting: SCIM autoloads with the initial use of gtk im module, so it is to
control the xinput behavior.

Comment 12 Bill Nottingham 2006-07-12 03:27:45 UTC
OK, has my ACK. Some comments:

'Use legacy' or 'use custom' should probably only be available if there are
legacy/custom IMs installed.

If you select 'Always use' with no SCIM installed, you get a dangling symlink.


Comment 13 Akira TAGOH 2006-07-12 03:31:58 UTC
yes, I'm aware of those issues and in my todo list actually FWIW.

Comment 14 Warren Togami 2006-07-12 19:29:41 UTC
1) "Always use input methods" and "Use custom input method" is redundant and
confusing.  It is unclear what the difference is between "Always" and selecting
the first option of custom.  Why not eliminate "Always" and make the pulldown
menu the top option?

2) Wouldn't it be best to hide "scim-qtimm" and "scim-bridge" because they are
not stand-alone, but rather additions to the environment set by "scim"?

Comment 15 Akira TAGOH 2006-07-13 00:45:35 UTC
BTW im-chooser has been approved for Extras now and new one is available on it.
"Always use input methods" option has been gone and it is now "Follow the
system-wide configuration" instead of. I still think that eliminating this
option isn't a good idea. technically "custom" makes a symlink to something
directly. it may be not good in some cases.

about 2), yes, Jens and I discussed about it to add the kind of tag in xinput
script to let im-chooser understand to hide from the option menu.

any comments about im-chooser are welcome. please try the latest version.

Comment 17 Jesse Keating 2006-07-19 16:55:06 UTC
Given it passed Extras review just shortly ago, approving for Core from a
package quality standpoint.

Adding with tagoh as owner.

You'll need to coordinate having it removed from Extras.

How should this be in Comps?

Comment 18 Akira TAGOH 2006-07-20 12:49:02 UTC
Well, ideally it should be installed by default regardless scim is installed or
not, to allow all people to setup their preference. Warren and I talked about it
a bit though, if someone enables IM on im-chooser without SCIM installed say,
getting the package through pirut say and installing them sounds good idea to me.
So I would push this to install by default for that purpose too.


Comment 19 Jesse Keating 2006-07-20 20:06:01 UTC
Added to Gnome and KDE desktop groups as a default.

Comment 20 Jesse Keating 2006-07-20 20:24:13 UTC
This has been built into rawhide.

Comment 21 Akira TAGOH 2006-07-24 03:36:47 UTC
BTW there is no component for im-chooser in bugzilla (I mean for Fedora Core).
can you add it?


Note You need to log in before you can comment on or make changes to this bug.