Closed
Bug 231750
Opened 21 years ago
Closed 21 years ago
Allow selection of text in gtk popup windows
Categories
(Core Graveyard :: Embedding: GTK Widget, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: crispin, Assigned: blizzard)
Details
Attachments
(1 file)
1.28 KB,
patch
|
blizzard
:
review+
bryner
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7a) Gecko/20040121 Galeon/1.3.11a
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7a) Gecko/20040121 Galeon/1.3.11a
The Gnome HIG recommends that text inside alert windows is selectable:
http://developer.gnome.org/projects/gup/hig/1.0/windows.html#alerts-information
This would allow people for instance to select text from javascript alert
messages etc.
Attached is a one-liner patch to make the gtk EmbedPrompter class have
selectable text.
See also bug 60513, which is the bug for selectable text in the XUL alerts.
Reproducible: Always
Steps to Reproduce:
Reporter | ||
Comment 1•21 years ago
|
||
Reporter | ||
Updated•21 years ago
|
Attachment #139601 -
Flags: review?(blizzard)
Reporter | ||
Updated•21 years ago
|
Attachment #139601 -
Flags: superreview?(bryner)
Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 139601 [details] [diff] [review]
Allow the label text to be selectable
Looks good to me.
Attachment #139601 -
Flags: review?(blizzard) → review+
Updated•21 years ago
|
Attachment #139601 -
Flags: superreview?(bryner) → superreview+
Reporter | ||
Comment 3•21 years ago
|
||
Thanks for the r= and sr=, now it has both, could someone please check in the patch.
Reporter | ||
Comment 4•21 years ago
|
||
This was checked in by neil, thanks :-)
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•