Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#4230 closed defect (fixed)

Order of menu items in palette differs from Gecko version

Reported by: humitos Owned by: humitos
Priority: Low Milestone:
Component: Browse Version: Unspecified
Severity: Minor Keywords: regression, r+, olpc-testing-pending
Cc: garycmartin, erikos Distribution/OS: OLPC
Bug Status: Unconfirmed

Description

I was told that the order of the items in the palette shown when you longpress on a link differs from the Gecko version of Browse.

Take a look at the screenshots.

Attachments (3)

gecko_browser.png (74.2 KB) - added by humitos 10 years ago.
webkit_browser.png (106.1 KB) - added by humitos 10 years ago.
0001-Order-of-MenuItems-in-palettes-SL-4230.patch (1.9 KB) - added by humitos 10 years ago.

Download all attachments as: .zip

Change History (12)

Changed 10 years ago by humitos

Changed 10 years ago by humitos

comment:1 follow-up: Changed 10 years ago by erikos

  • Cc garycmartin added
  • Priority changed from Normal to Low
  • Severity changed from Unspecified to Minor

Gary, any specific order you favour?

comment:2 in reply to: ↑ 1 ; follow-up: Changed 10 years ago by garycmartin

Replying to erikos:

Gary, any specific order you favour?

I don't think there is a strong UI argument for a particular order here, but I would suggest going back to the original order to preserve existing documentation, screen shots, user memory. Don't change things we don't have a good reason for changing ;)

comment:3 in reply to: ↑ 2 Changed 10 years ago by erikos

Replying to garycmartin:

Replying to erikos:

Gary, any specific order you favour?

I don't think there is a strong UI argument for a particular order here, but I would suggest going back to the original order to preserve existing documentation, screen shots, user memory. Don't change things we don't have a good reason for changing ;)

Perfect, so Humitos, please get back to the old ordering. Thanks.

comment:4 Changed 10 years ago by humitos

  • Owner changed from manuq to humitos
  • Status changed from new to accepted

comment:5 Changed 10 years ago by humitos

  • Cc erikos added
  • Keywords regression r? added

comment:6 follow-up: Changed 10 years ago by manuq

Thanks, patch looks good. What about the order of the palette for the images? Can you check if that one is inverted as well?

comment:7 in reply to: ↑ 6 Changed 10 years ago by humitos

Replying to manuq:

Thanks, patch looks good. What about the order of the palette for the images? Can you check if that one is inverted as well?

I've already checked that. It's correct

comment:8 Changed 10 years ago by erikos

  • Keywords r+ olpc-testing-pending added; r? removed
  • Resolution set to fixed
  • Status changed from accepted to closed

comment:9 Changed 9 years ago by dnarvaez

  • Milestone 0.98 deleted

Milestone 0.98 deleted

Note: See TracTickets for help on using tickets.