Opened 10 years ago

Closed 9 years ago

Last modified 20 months ago

#2006 closed defect (fixed)

Palette visual style specification not matched

Reported by: Eben Owned by: edsiper
Priority: high Milestone:
Component: sugar Version:
Keywords: Cc: marco, benzea
Blocked By: Blocking:
Deployments affected: Action Needed:
Verified: no


Rollovers should be 75px high. The text inside them should be vertically centered, and aligned 15px from the left edge.

Also, the rollover text seems to have a gray rollover state itself; this should be removed.

Attachments (1)

palette_spec.png (42.9 KB) - added by Eben 10 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by Eben

  • Cc marco added

I want to add some details to this spec. The primary rollover should be 75px high; the secondary should be as high as needed. Both should be the width required for the secondary.

The primary rollover needs to support 4 parameters: primary title, secondary title, and icon, shortcut. The primary title is the only required parameter. They can be specified in any combination, with the exception that you can't use a secondary title in conjunction with a shortcut; you could just drop the secondary title if both are specified. The primary title should be bold.

The primary and secondary titles should be left aligned, and when there is no icon there should be 15px padding on the left. Likewise, the shortcut should be right aligned, with 15px right padding. When an icon is specified, it should be centered within the 75px block with no additional padding, and the text should be left aligned at the 75px mark (from left edge of palette). We probably also need an easy way to specify the shortcuts via API, akin to shortcut('k', SHIFT | ALT | CTRL), such that the character (always capitalized) and the glyphs for the modifiers specified by the bit flags get arranged in a consistent order (SHIFT ALT CTRL K).

Most contents within the rollover should be padded on all sides with the same 15px margin. Note, however, that this margin needs to be a guideline, not a hard enforced rule. It's up to the creator of the palette to adhere to this where appropriate, as some controls such as the description text field background, tabs, grouping controls, etc. may stretch to the edge.

I'm attaching an image which illustrates several of the possible states. These should be matched as exactly as possible.

Changed 10 years ago by Eben

comment:2 Changed 10 years ago by Eben

  • Summary changed from Rollover visual style incorrect to Palette visual style specification not matched

comment:3 Changed 10 years ago by Eben

  • Cc benzea added
  • Milestone changed from First Deployment to Trial-3

I'm bumping this up since this is such a primary interface element. This affects both the OS and almost all activities, so it's important to get this as close to correct as possible. As soon as these are rendered cleanly, the look of the entire laptop will improve dramatically.

Let's get some feedback loops in place around the palettes, in terms of basic implementation, styling, and API. How far are we from reaching the target spec (image attached)?

comment:4 Changed 10 years ago by Eben

  • Priority changed from normal to high

An addendum to this spec: We should institute a minimum width policy for palettes so they don't look as silly. I'm thinking that a minimum of 150px would do the trick. This will also apply to things like group view and the calc activity etc. in the frame, where the primary palette is actually smaller (currently) in both width and height than the object itself. I think an absolute minimum of (150 x 75px ) is a good one to keep things looking normal.

comment:5 Changed 9 years ago by marco

  • Resolution set to fixed
  • Status changed from new to closed

The spec is moved on the wiki, we implemented part of it and there are tickets for some of the stuff which is not implemented. Marking fixed, feel free to open separate tickets about other aspects that you think should be fixed soon.

comment:6 Changed 20 months ago by Quozl

  • Milestone Trial-3 deleted

Milestone Trial-3 deleted

Note: See TracTickets for help on using tickets.