unity-2d

[dash] Rating stars in right-to-left locales behave inconsistently.

Bug #945719 reported by Haggai Eran
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity-2d
Fix Released
Low
Haggai Eran
unity-2d 5.7
unity-2d (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

When using a right-to-left locale, the rating stars in the dash (e.g. in the applications lens) are displayed mirrored (this is the expected rendering, except for the fact that they are aligned to the left, instead of to the right).
However, when clicking a star, instead of marking all the stars from the rightmost star to the currently clicked star, the widget behaves as if the click was onto a left-to-right locale. For instance, if the second star from the left is clicked, then the two right-most stars are marked. This example is captured in the attachement. After clicking at the mouse position in the screenshot, which is at the second star from the left, the two right-most stars are marked.

Tags: rtl
Revision history for this message
Haggai Eran (haggai-eran) wrote :
tags: added: rtl
Haggai Eran (haggai-eran)
Changed in unity-2d:
status: New → In Progress
Revision history for this message
Haggai Eran (haggai-eran) wrote :

Hi,

This attachment show how the widget looks after applying the patch at lp:~haggai-eran/unity-2d/rtl-rating-stars-945719.

Unity Merger (unity-merger)
Changed in unity-2d:
status: In Progress → Fix Committed
Gerry Boland (gerboland)
Changed in unity-2d:
milestone: none → 5.8
assignee: nobody → Haggai Eran (haggai-eran)
importance: Undecided → Low
Didier Roche-Tolomelli (didrocks)
Changed in unity-2d:
milestone: 5.8 → 5.7
Didier Roche-Tolomelli (didrocks)
Changed in unity-2d:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity-2d - 5.7.0-0ubuntu1

---------------
unity-2d (5.7.0-0ubuntu1) precise; urgency=low

  * New upstream release:
    - unity-2d-panel crashed with SIGABRT in Unity2dDebug::unity2dQtHandler()
      (LP: #925457)
    - unity-2d-shell crashed with SIGSEGV in ref() (LP: #947278)
    - Launcher - Make Launcher left of screen reveal more responsive and less
      prone to false positives (LP: #765819)
    - [dash] [launcher] [panel] Tinting based on wallpaper missing
      (LP: #882101)
    - [launcher] In multimonitor setup Unity-2D only shows launcher on one
      monitor (fails MM UX spec #2.8) (LP: #780204)
    - [unity-2d] FF exception to add PointerBarrier to Unity2d (LP: #947976)
    - multimonitor, launcher: Provide an option to display either a single
      launcher or a launcher on each display in a multi-monitor environment
      (LP: #950136)
    - lens switching is not keyboard navigable (LP: #945220)
    - [panel] [multi-monitor] On secondary screen, unable to scrub from
      indicator menu to app menu (LP: #844809)
    - "Workspaces" button displays previous wallpaper (LP: #900019)
    - Dash - cursor should change when pointer is over the search field
      (LP: #849175)
    - Filter checkboxes are without margins in right to left locales
      (LP: #949319)
    - [panel] [multi-monitor] Open indicator menu on one screen causes other
      indicator to look open too (LP: #844798)
    - Dash - horizental divider line in between categories incorrectly drawn
      (LP: #841750)
    - [launcher] Tile Progress bar starting from left of launcher, not
      container (LP: #912777)
    - Dash - dash is not closed with alt+f4 (LP: #891818)
    - [dash] Rating stars in right-to-left locales behave inconsistently.
      (LP: #945719)
    - LensButton.qml:72 warning when running unity-2d (LP: #947276)
    - [dash] Multi-range filters are rendered incorrectly in right-to-left
      locales (LP: #950649)
  * debian/unity-2d.gconf-defaults:
    - removed: not used anymore and causing regression with hide-mode=2
      by default, converted to the gsettings key (LP: #942772)
  * debian/unity-2d.preinst:
    - removed, was a natty -> oneiric transition
  * debian/control, debian/unity-2d-common.install:
    - create unity-2d-common and make some packages depending on it
    - move some files from unity-2d package to unity-2d-common
      (gsettings and apport hook) to have packaging shipping their schema
      (LP: #925457)
  * debian/unity-2d-common.install.in:
    - no more gconf -> gsettings migration shipped
  * debian/unity-2d.postinst, prerm, trigger:
    - moved to unity-2d-common new package, to ensure even people uninstalling
      unity-2d got the right metacity configuration in the session
    - update the trigger to the correct directory
  * debian/control:
    - bump to latest Standards-Version
    - fix some too long description
    - put the transitional packages in the oldlibs/extra section
    - build-dep on libxtst-dev, libxfixes-dev for barrier change
    - need latest metacity for barriers to work
 -- Didier Roche <email address hidden> Mon, 19 Mar 2012 14:07:10 +0100

Changed in unity-2d (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.