Closed
Bug 421410
Opened 15 years ago
Closed 15 years ago
selected text in input box is cleared/unselected after print preview
Categories
(Core :: Print Preview, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9beta5
People
(Reporter: wsmwk, Assigned: martijn.martijn)
Details
Attachments
(1 file)
2.79 KB,
patch
|
enndeakin
:
review+
beltzner
:
approval1.9+
|
Details | Diff | Splinter Review |
selected text in input box is cleared/unselected after printing (print preview is OK) 1. select any text typed into bugzilla description field 2. file|print Expected results: text still selected Actual results: text selection has been cleared/unselected Similar to Bug 304313 - selected text cleared/unselected after print preview. So perhaps whatever was used to fix that would help this? Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b5pre) Gecko/2008030506 Minefield/3.0b5pre
Assignee | ||
Comment 1•15 years ago
|
||
This happens because the focus isn't returned to the text input after print preview, see discussion in bug 128953 as to why you don't see a selection when the input is blurred. So this bug is basically about "Focus isn't returned to the element that had focus before print preview".
Assignee | ||
Comment 2•15 years ago
|
||
And that should probably be fixed somewhere around here: http://lxr.mozilla.org/seamonkey/source/toolkit/components/printing/content/printUtils.js#261
Assignee | ||
Comment 3•15 years ago
|
||
Oops, I misread the bug, it's actually working for you for print preview, which is strange, because it's not working for me on print preview, but it's working fine for me with printing.
Reporter | ||
Comment 4•15 years ago
|
||
yeah, odd. I thought I was pretty careful in writing it up - but what I'm seeing now is what you say. So I got it backward - print OK, preview not. interesting when using _print_ if a dialog is not not on screen - the intitial print dialog and the progress dialog- then the selection is visible but when dialog is up the selection is not visible - is that what you are saying in comment 1?
Status: UNCONFIRMED → NEW
Component: Printing → Print Preview
Ever confirmed: true
Summary: selected text in input box is cleared/unselected after printing → selected text in input box is cleared/unselected after print preview
Assignee | ||
Comment 5•15 years ago
|
||
Yeah, that's what I was saying (sort of). The selection of text controls is made invisible when the focus moves away from it.
Assignee | ||
Comment 6•15 years ago
|
||
This fixes it. It basically makes use of the code in bug 323805. I couldn't find a Seamonkey version of printUtils.js, perhaps it uses the same file, I don't know. I guess it might be better if gFocusedElement is a member of PrintUtils, right?
Attachment #308024 -
Flags: review?(enndeakin)
Comment 7•15 years ago
|
||
Comment on attachment 308024 [details] [diff] [review] patch Seems OK
Attachment #308024 -
Flags: review?(enndeakin) → review+
Assignee | ||
Updated•15 years ago
|
Attachment #308024 -
Flags: approval1.9?
Comment 8•15 years ago
|
||
Comment on attachment 308024 [details] [diff] [review] patch a1.9=beltzner
Attachment #308024 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → martijn.martijn
Keywords: checkin-needed
Comment 9•15 years ago
|
||
Checking in toolkit/components/printing/content/printUtils.js; /cvsroot/mozilla/toolkit/components/printing/content/printUtils.js,v <-- printUtils.js new revision: 1.31; previous revision: 1.30 done
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta5
Version: unspecified → Trunk
You need to log in
before you can comment on or make changes to this bug.
Description
•