Closed Bug 421410 Opened 15 years ago Closed 15 years ago

selected text in input box is cleared/unselected after print preview


(Core :: Print Preview, defect)

Windows XP
Not set





(Reporter: wsmwk, Assigned: martijn.martijn)



(1 file)

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
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".
And that should probably be fixed somewhere around here:
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.
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?
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
Yeah, that's what I was saying (sort of). The selection of text controls is made invisible when the focus moves away from it.
Attached patch patchSplinter Review
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 on attachment 308024 [details] [diff] [review]

Seems OK
Attachment #308024 - Flags: review?(enndeakin) → review+
Attachment #308024 - Flags: approval1.9?
Comment on attachment 308024 [details] [diff] [review]

Attachment #308024 - Flags: approval1.9? → approval1.9+
Assignee: nobody → martijn.martijn
Keywords: checkin-needed
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
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.