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 117013 - [PATCH] Assistive Technologies Dialog isn't HIGified
[PATCH] Assistive Technologies Dialog isn't HIGified
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Other Preferences
2.3.x
Other All
: High normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks: 115435
 
 
Reported: 2003-07-08 18:55 UTC by Christian Neumair
Modified: 2003-07-17 06:31 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Proposed patch against HEAD. (12.47 KB, patch)
2003-07-08 18:56 UTC, Christian Neumair 		none Details | Review
Screenshot of the new dialog layout (31.93 KB, image/png)
2003-07-08 18:58 UTC, Christian Neumair 		  Details

Description Christian Neumair 2003-07-08 18:55:02 UTC 
Incoming.

regs,
 Chris
Comment 1 Christian Neumair 2003-07-08 18:56:36 UTC 
Created attachment 18143 [details] [review]
Proposed patch against HEAD.
Comment 2 Christian Neumair 2003-07-08 18:58:16 UTC 
Created attachment 18144 [details]
Screenshot of the new dialog layout
Comment 3 Christian Neumair 2003-07-08 19:03:11 UTC 
Changing some bug data.

regs,
 Chris
Comment 4 Christian Neumair 2003-07-09 15:43:28 UTC 
Here is what I did:
- Remove separator
- Remove useless padding from child widgets
- Move widget spacing to container properties
- Change spacing to HIG-compliant values
- change "Start these assistive technologies every time you log in:"
sensitivity together with all the other widgets in its category. It
used to be ignored.

regs,
 Chris
Comment 5 Calum Benson 2003-07-15 17:45:49 UTC 
Changes look fine to me, marking as ui-review comments so they'll get
into 2.4 hopefully...
Comment 6 Dennis Cranston 2003-07-17 06:31:32 UTC 
Sorry to step on your toes Chris.  I checked in a patch to CVS to do
the same thing. :-( 

I am resolving this bug report as fixed.