Closed Bug 355208 Opened 18 years ago Closed 18 years ago

Range leaves artifacts

Categories

(Core Graveyard :: XForms, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aaronr, Assigned: aaronr)

References

Details

(Keywords: fixed1.8.0.8, fixed1.8.1.1)

Attachments

(2 files, 1 obsolete file)

(deleted), application/xhtml+xml
Details
(deleted), patch
surkov
: review+
smaug
: review+
Details | Diff | Splinter Review
Assume you have a default range and an incremental range bound to the same node. Set the value of the default range to a new value. Click on the incremental range to set a third different value. There will be an artifact left on the incremental range. The problem is that when the focus changes from the non-incremental to incremental range due to the mouse click, the bound node will get the value of the non-incremental range. The incremental change will then get the value of the bound node. When the mouse down event is handled, the slider is erased from the position of prevX. However, prevX will reflect the INITIAL value of the incremental slider, not the current value, so we are erasing a slider that doesn't exist and we leave around the current slider.
Attached file testcase (deleted) —
Attached patch patch 1 (obsolete) (deleted) — Splinter Review
fixed this by setting prevX to null on mouseup and upon initialization. And at mousedown when prevX isn't set, then I initialize it to the x position that corresponds to the current slider value.
Attachment #241034 - Flags: review?(surkov.alexander)
Looks right, do you need in "dump" calls and should you null prevX on 'mouseup' event (I doubt since 'handler' object is recreated for every 'mousedown' event)? Today later I'll look more precisely.
Attached patch patch 2 (deleted) — Splinter Review
Sorry, I should have made sure that I had all of the dumps out of there. You are also right in that we don't need to set it to null on mouseup.
Attachment #241034 - Attachment is obsolete: true
Attachment #241034 - Flags: review?(surkov.alexander)
Attachment #241046 - Flags: review?(surkov.alexander)
Attachment #241046 - Flags: review?(surkov.alexander) → review+
Blocks: 334603
Status: NEW → ASSIGNED
checked into trunk
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch
checked into 1.8.0 branch on 2006/10/11
Keywords: fixed1.8.0.8
checked into 1.8 branch on 2006/11/21
Keywords: fixed1.8.1.1
Whiteboard: xf-to-branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: