Clicking scrollbutton in <select> listbox scrolls by two <option>s

RESOLVED FIXED

Status

()

Core
Widget: Cocoa
--
minor
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: Jesse Ruderman, Assigned: Josh Aas)

Tracking

({regression, testcase})

Trunk
x86
Mac OS X
regression, testcase
Points:
---
Bug Flags:
blocking1.9 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

11 years ago
Created attachment 266326 [details]
testcase

Steps to reproduce:
1. Load the testcase.
2. Click the down scrollbutton.

Result: scrolls by two options.

Expected: should scroll by one option.
Flags: blocking1.9?
(Assignee)

Comment 1

10 years ago
Can anyone reproduce this on Windows or Linux?

Comment 2

10 years ago
WFM with Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a8pre) Gecko/2007091104 Minefield/3.0a8pre
also wfm on Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.9a8pre) Gecko/2007091005 Minefield/3.0a8pre ID:2007091005
(Assignee)

Updated

10 years ago
Flags: blocking1.9? → blocking1.9+
(Assignee)

Comment 4

10 years ago
Created attachment 284430 [details] [diff] [review]
fix v1.0

Dave Hyatt added code that messed this up in bug 129893 a long time ago. We should just kill it off.
Attachment #284430 - Flags: review?(cbarrett)
Does this mean scrolling speed is decreased? We already scroll slower than Safari does.
(Assignee)

Comment 6

10 years ago
I don't think so. And it doesn't really matter, we can't be doing what is done in that ifdef anyway.
(Assignee)

Comment 7

10 years ago
Hyatt added a similar thing in presshell, see the "#ifdef MOZ_WIDGET_COCOA" there. I think getting rid of that might actually change scrolling speed in a way that matters, but this shouldn't.
Comment on attachment 284430 [details] [diff] [review]
fix v1.0

Ah, that's what I was thinking of when I read this patch, then.
Attachment #284430 - Flags: review?(cbarrett) → review+
(Assignee)

Updated

10 years ago
Attachment #284430 - Flags: superreview?(roc)
Attachment #284430 - Flags: superreview?(roc) → superreview+
(Assignee)

Comment 9

10 years ago
landed on trunk
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.