editable contents of js prompt sheet may line wrap with no visual indication given

RESOLVED FIXED in Camino0.9

Status

RESOLVED FIXED
16 years ago
14 years ago

People

(Reporter: chris, Assigned: mikepinkerton)

Tracking

({testcase})

unspecified
Camino0.9
PowerPC
macOS
testcase

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

16 years ago
Camino 2003080402 & many previous builds.

(bug became known to me while using the 'shorten bug query' bookmarklet here:
http://www.squarefree.com/bookmarklets/mozilla.html )

Long strings or long passages can be entered into a javascript prompt and when
this happens instead of staing on one line and folowing off the screen
horizontally like most UI elements (url bar, text fields, etc) this sheet seems
to want to word wrap.

In the case of a pre-populated dialog like the included where the user doesn't
know what the "default" content this may cause a number of problems may occur
including data loss or just erroneous data being submitted to an application.


Apparent content of prompt:

http://bugzilla.mozilla.org/

Actual content of prompt:
http://bugzilla.mozilla.org/buglist.cgi?product=Firebird&keywords=regression&votes=1&resolution=---&order=bugs.votes,
bugs.bug_id

To access the entire content care must be given to select all and copy paste
elsewhere or know to hit the down arrow to see the other 3 'lines' of text.

Resolution:
Either keep the content all on on one line as other elements and other browsrs
do (providing the visual clue of having overrunning text) or make the prompt
area a multi-line <html:textarea> like widget with scrollers and other visual
clues that there is more data.

Quick screen shot will be attached in a second.
(Reporter)

Comment 1

16 years ago
heres a quick shot of the prompt used in this example... it looks almost
identical to the output of: javascript:prompt("Copy this:",
"http://bugzilla.mozilla.org/");
curious, i'll take a look
Status: NEW → ASSIGNED
Target Milestone: --- → Camino1.0
I'm thinking the text field just needs to be changed from "wraps" to "scrolls".
Wevah: Care to test that and possibly submit a patch?
Posted patch Probable patch! (obsolete) — Splinter Review
This should do it, if I got the file right. Someone should test this to make
sure.
Oops, wrong method. Should use setScrollable:YES and not setWraps:NO.

Sorry for the quasi-spam.
Attachment #191036 - Attachment is obsolete: true

Updated

14 years ago
Attachment #191036 - Attachment filename: nsnsAlertController.patch → nsAlertController.patch
landed, thanks!
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Target Milestone: Camino1.0 → Camino0.9
You need to log in before you can comment on or make changes to this bug.