Use a native rendering for meter elements on MacOS X

RESOLVED FIXED in mozilla16

Status

()

Core
Widget: Cocoa
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: mounir, Assigned: mounir)

Tracking

(Blocks: 1 bug)

Trunk
mozilla16
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

6 years ago
Created attachment 538077 [details] [diff] [review]
WIP Patch

I see three open questions:
 - What are the native sizes (for mini, small, regular widgets)?
 - Is there a native vertical style?
 - There are 4 styles, the most obvious one to use by default is continuous but do we want to allow the authors to use discrete, rating and relevancy ones? I would say not for the moment. Webkit allows that, see http://trac.webkit.org/wiki/Styling%20Form%20Controls

Work in progress patch attached works but don't use native sizes (but progress ones) and don't takes into account vertical widget.
(Assignee)

Comment 1

6 years ago
Created attachment 538255 [details] [diff] [review]
Patch v1

There are three widget sizes (mini, small and regular) but all of them have the same size (height=16px).
I will write another patch for the vertical meter widget given that there is no native widget for that and I will have to rotate the widget in the implementation.
Attachment #538077 - Attachment is obsolete: true
Attachment #538255 - Flags: review?(joshmoz)
(Assignee)

Updated

6 years ago
Blocks: 663119
(Assignee)

Updated

6 years ago
Whiteboard: [needs review]
(Assignee)

Comment 2

6 years ago
Created attachment 538317 [details] [diff] [review]
Patch v1

Adding a test for RTL meter.
Attachment #538255 - Attachment is obsolete: true
Attachment #538255 - Flags: review?(joshmoz)
Attachment #538317 - Flags: review?(joshmoz)
(Assignee)

Updated

6 years ago
Depends on: 663365

Updated

6 years ago
Attachment #538317 - Flags: review?(joshmoz) → review?(bgirard)

Comment 3

6 years ago
Benoit, can you do this review?
Comment on attachment 538317 [details] [diff] [review]
Patch v1

Review of attachment 538317 [details] [diff] [review]:
-----------------------------------------------------------------

r+
Attachment #538317 - Flags: review?(bgirard) → review+
(Assignee)

Updated

6 years ago
Whiteboard: [needs review] → [ready to land][waits for dependencies]
(Assignee)

Comment 5

5 years ago
https://hg.mozilla.org/mozilla-central/rev/cd74e9324088
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [ready to land][waits for dependencies]
Target Milestone: --- → mozilla16
You need to log in before you can comment on or make changes to this bug.