Ticket #1544 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

Sugar gconf settings breaks mouse buttons behaviour in gnome session

Reported by: alsroot Owned by: tomeu
Priority: Unspecified by Maintainer Milestone: 0.88
Component: sugar Version: Git as of bugdate
Severity: Critical Keywords: r+
Cc: icarito Distribution/OS: Ubuntu
Bug Status: Assigned

Description

|TestCase|
* login to default gnome session(e.g. to new account)
* default mouse behaviour with key modifier pressed is, press <Alt> + mouse clicks to do some window related stuff
* start sugar-emulator
* out of sugar-emulator window, mouse behaviour looks like modifier was pressed(but it doesn't) to disable it, pressing <Alt> it required

Work flow is not reproducible in all cases. I got a chance to setup karmic on new notebook and got 100% of reproducibility of this issue but had to return notebook and didn't investigate issue(but found that fallback gnome session "resolves" issue). In all my current environments(VM and natinve) I can't reproduce this bug.

Change History

  Changed 4 years ago by alsroot

tuukkah on #sugar said,
hmm, i've seen the same, running metacity --replace helped

could someone who can reproduce this issue run metacity with "--replace" argument(add it to emulator.py)

follow-up: ↓ 4   Changed 4 years ago by alsroot

looks like its a composite WM issue in gnome, pure metacity works fine and only adding composite effects reveals this issue

  Changed 4 years ago by RafaelOrtiz

  • milestone changed from Unspecified by Release Team to 0.88

in reply to: ↑ 2 ; follow-up: ↓ 5   Changed 4 years ago by icarito

Replying to alsroot:

looks like its a composite WM issue in gnome, pure metacity works fine and only adding composite effects reveals this issue

Actually, pure metacity does have problems, only the symptom is different.
With compiz, <Alt> key seems "stuck".
Without compiz - <Alt> modifier key for moving windows around, does not work.

in reply to: ↑ 4 ; follow-ups: ↓ 6 ↓ 8   Changed 4 years ago by alsroot

Replying to icarito:

Replying to alsroot:

looks like its a composite WM issue in gnome, pure metacity works fine and only adding composite effects reveals this issue


Actually, pure metacity does have problems, only the symptom is different.
With compiz, <Alt> key seems "stuck".

yup, thats what I meant by "adding composite effects"

Without compiz - <Alt> modifier key for moving windows around, does not work.

thats known issue for 0.86(its hard to say how to isolate gconf settings that sugar rewrites)

I'm thinking about removing gconf reseting of key modifier in jhconvert distros, since sugar makes gnome compozite session unuseful.

in reply to: ↑ 5   Changed 4 years ago by icarito

  • cc icarito added

  Changed 4 years ago by icarito

  • status_field changed from Unconfirmed to Assigned

Confirmed, also happens on Trisquel.
Workaround: Having a separate user for Sugar.

in reply to: ↑ 5 ; follow-up: ↓ 9   Changed 4 years ago by tomeu

Replying to alsroot:


I'm thinking about removing gconf reseting of key modifier in jhconvert distros, since sugar makes gnome compozite session unuseful.

Can you extend on what you mean by that? And what would be a longer term fix?

in reply to: ↑ 8 ; follow-up: ↓ 10   Changed 4 years ago by alsroot

Replying to tomeu:

Replying to alsroot:


I'm thinking about removing gconf reseting of key modifier in jhconvert distros, since sugar makes gnome compozite session unuseful.


Can you extend on what you mean by that? And what would be a longer term fix?

do you really need changing key modifier?
I guess in current situation we can't do something useful from sugar side.

in reply to: ↑ 9   Changed 4 years ago by alsroot

Replying to alsroot:

Replying to tomeu:

Replying to alsroot:


I'm thinking about removing gconf reseting of key modifier in jhconvert distros, since sugar makes gnome compozite session unuseful.


Can you extend on what you mean by that? And what would be a longer term fix?


do you really need changing key modifier?

oops, s/you/we/

I guess in current situation we can't do something useful from sugar side.

  Changed 4 years ago by tomeu

Don't remember what "changing key modifier" means, can you explain? Thanks!

follow-up: ↓ 13   Changed 4 years ago by alsroot

sugar changes /apps/metacity/general/mouse_button_modifier
on startup, so do we really need to change existed behaviour,
in my mind situation when user holding ALT modifier can move metacity windows is unusual

At least in 0.86 we leave this gconf setting as is, and ping upstream to fix this bug for 0.88 cycle.

in reply to: ↑ 12 ; follow-up: ↓ 14   Changed 4 years ago by tomeu

Replying to alsroot:

sugar changes /apps/metacity/general/mouse_button_modifier
on startup, so do we really need to change existed behaviour,
in my mind situation when user holding ALT modifier can move metacity windows is unusual

Kids always manage to surprise us :/

At least in 0.86 we leave this gconf setting as is, and ping upstream to fix this bug for 0.88 cycle.

Upstream is not very actively maintained, do you have an idea of what would take a proper fix?

in reply to: ↑ 13   Changed 4 years ago by alsroot

Replying to tomeu:

Replying to alsroot:

At least in 0.86 we leave this gconf setting as is, and ping upstream to fix this bug for 0.88 cycle.


Upstream is not very actively maintained, do you have an idea of what would take a proper fix?

didn't yet look to code, but guess it shouldn't be hard to fix, just changing reaction on gconf value, will look

  Changed 3 years ago by alsroot

upstream bug  https://bugzilla.gnome.org/show_bug.cgi?id=570183

the workaround is using e.g. <Super> value for /apps/metacity/general/mouse_button_modifier since it shouldn't interfere with out Alt bindings

  Changed 3 years ago by alsroot

Workaround which is used in jhconvert packages
 http://git.sugarlabs.org/projects/jhconvert/repos/mainline/blobs/master/patches/0.86/1544.patch

  Changed 3 years ago by lfaraone

  • keywords r? added
  • version changed from 0.86.x to Git as of bugdate

follow-ups: ↓ 19 ↓ 20   Changed 3 years ago by tomeu

  • keywords r! added; r? removed

I'm quite confused at how alsroot's workaround can work. Just tried it here and with <Super> I cannot move windows with <Alt>+Click.

That said, I don't see how it could be a problem for Sugar, so I can apply that patch if it works for the rest.

in reply to: ↑ 18   Changed 3 years ago by alsroot

Replying to tomeu:

I'm quite confused at how alsroot's workaround can work. Just tried it here and with <Super> I cannot move windows with <Alt>+Click.

The problem can be reproducible only with enabling composite effects in metacity. Just enable it in gnome, then login to sugar, then back to gnome. In gnome with enabled composite effects and 'Disabled' in mouse_button_modifier gconf key, mouse behaves like Ctrl is pressed. Maybe it was fixed in recent metacity but anyway there are desktops in field that will be affected by such ugly(in case of regular desktops, i.e., with switching between gnome/sugar) bug for some time.

That said, I don't see how it could be a problem for Sugar, so I can apply that patch if it works for the rest.

in reply to: ↑ 18   Changed 3 years ago by alsroot

Replying to tomeu:

That said, I don't see how it could be a problem for Sugar, so I can apply that patch if it works for the rest.

It's about that one login to sugar makes gnome session(with composite metacity) broken.

  Changed 3 years ago by tomeu

  • keywords r+ added; r! removed

Sounds good then.

  Changed 3 years ago by alsroot

  • status changed from new to closed
  • resolution set to fixed

 http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/c5ac2887e7cb121645747d339f7b34dc8d673992

Note: See TracTickets for help on using tickets.