Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not set the cursor globally #670

Closed
wants to merge 0 commits into from
Closed

Conversation

erilyth
Copy link
Contributor

@erilyth erilyth commented Apr 1, 2016

Issue where the cursor is not being reset even when we exit 'Sugar'.
Discussed on mailing list - '[Sugar-devel] Mouse pointer changes into sugar style pointer in ubuntu'

@jvonau
Copy link
Contributor

jvonau commented Apr 1, 2016

Some background ea803cb

What if the user's starting 'cursor-theme' was not 'default', this would alter that user's choice of themes. Might want to read the current setting, save it and use the observed value to restore the user's previous setting in place of using 'default'.

@erilyth
Copy link
Contributor Author

erilyth commented Apr 1, 2016

@jvonau, It now gets reset to the user's original cursor.

@jvonau
Copy link
Contributor

jvonau commented Apr 1, 2016

should address inadvertent changing of a user setting, currently untested by myself.

@quozl
Copy link
Contributor

quozl commented Apr 1, 2016

Would like also to support running /usr/bin/sugar in GNOME or Unity desktop; the cursor should be private to Sugar and activities, not infect all other desktop apps.

@erilyth
Copy link
Contributor Author

erilyth commented Apr 2, 2016

@quozl ,how do you think I should proceed to get that done?

@quozl
Copy link
Contributor

quozl commented Apr 4, 2016

@erilyth, look for a more subtle way to set the cursor. Changing GSettings is overkill, and dates from the time that Sugar was alone on a device.

@davelab6
Copy link
Contributor

davelab6 commented Apr 4, 2016 via email

@quozl
Copy link
Contributor

quozl commented Apr 4, 2016

@davelab6, i'm not even sure who does moderation duties on bugs.sugarlabs.org.

@davelab6
Copy link
Contributor

davelab6 commented Apr 4, 2016

@davelab6, i'm not even sure who does moderation duties on bugs.sugarlabs.org.

Aha! :)

@walterbender do you know who does moderate bugs.sugarlabs.org?

When I submit tickets it says,

Moderation required for new ticket.

Thank you for submitting a new ticket ("Create sugar-event-recorder-daemon package as required part of sugar desktop").

Your submission has been routed to Samuel Cantero for approval. We make every effort to review submissions within one working day.

If you provided a valid e-mail address with your submission, you will receive e-mail notification if the moderator accepts your submission.

@quozl
Copy link
Contributor

quozl commented Apr 4, 2016

Ah, okay, thanks. Not being subject to moderation means I'm ignorant. Samuel Cantero is regularly responsive on systems@ mailing list. (fragmentation again).

@davelab6
Copy link
Contributor

davelab6 commented Apr 4, 2016

Okay cool, I'll post there and CC him as I found his email :)

@erilyth
Copy link
Contributor Author

erilyth commented Apr 4, 2016

@quozl There are ways to set the cursor using Gdk, but the problem is that the cursor needs to be set individually for every activity. Since the cursor is set on a given window. Would this be okay or should I look for something better?

@quozl
Copy link
Contributor

quozl commented Apr 4, 2016

@erilyth, that's great, sounds like what we need. So you'll be able to put that in sugar-toolkit-gtk3 in the activity common code. Once you show us that, I can look at whether I can backport it to sugar-toolkit (gtk2).

@erilyth erilyth force-pushed the master branch 2 times, most recently from b6f54cf to bf9af6e Compare April 5, 2016 09:22
@erilyth
Copy link
Contributor Author

erilyth commented Apr 5, 2016

@quozl , I'll send a PR to sugar-toolkit-gtk3

@erilyth erilyth changed the title Reset cursor to default on exit Do not set the cursor globally Apr 5, 2016
@erilyth erilyth closed this Apr 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants