sugarlabs / sugar

Sugar GTK shell
GNU General Public License v3.0
255 stars 241 forks source link

[Feature]: Add accessibility keys to Journal #791

Closed rhl-bthr closed 2 years ago

rhl-bthr commented 6 years ago

Feature: https://bugs.sugarlabs.org/ticket/4891

This is an update to the existing PR #667

Updates done (requested in #667):

Other Updates:

Could not be fixed:

@quozl, Kindly review

rhl-bthr commented 6 years ago

Update on the PR:

rhl-bthr commented 6 years ago

Also requesting others to review and suggest more improvements

rhl-bthr commented 6 years ago

Update: Commit message updated to detect co-author @iamutkarshtiwari in 2670767

@quozl, is this change approproate

quozl commented 6 years ago

Tested e429e95 on Ubuntu 16.04.

Curious behaviour; can you reproduce this;

Expected result: nothing.

Observed result: area below list is shaded grey. screenshot_ubuntu-xenial_2018-03-15_15 08 48

rhl-bthr commented 6 years ago

I am able to reproduce the effect. I remember seeing this effect in one of the activities as well, not sure which.

Another way to reproduce the same effect is:

I will try to find the cause and a fix for this

rhl-bthr commented 6 years ago

Update: The shading of the bottom empty area:

I am unable to find a cause or a fix for this. The area gets shaded when the last entry is in focus and then loses focus. Requesting help from developers on the same.

quozl commented 6 years ago

Requesting help from developers on the same.

Developers of GTK+ may be helpful. Write a stand-alone program that reproduces the problem, then iteratively remove as much as possible from the program until the program is as small as possible yet still reproduces. Then post a pointer to the program to a GTK+ mailing list or issue tracker and ask for help.

chimosky commented 2 years ago

@quozl could you share the branch this work is being done on if you still have it? The branch has been deleted by the author.

jcameron-sso commented 2 years ago

You can still do this;

git fetch origin pull/791/head
git checkout abc62125c

therefore the branch being deleted is of no consequence.

chimosky commented 2 years ago

You can still do this;

git fetch origin pull/791/head
git checkout abc62125c

therefore the branch being deleted is of no consequence.

Thanks.

chimosky commented 2 years ago

Moved to #959.