Slider's tooltip too small sometimes

Bug #1228575 reported by Roman Shchekin
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu Shorts App
Fix Released
Medium
Unassigned
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Medium
Nekhelesh Ramananthan
Vivid
New
Undecided
Unassigned

Bug Description

I tried to reimplement Slider's formatValue function this way:

function formatValue(v) {
                            if (v < maximumValue / 3)
                                return i18n.tr("Small")
                            else if (v < maximumValue / 3 * 2)
                                return i18n.tr("Midium")
                            else return i18n.tr("Large")
                        }

Result: word "Medium" doesn't fit in tooltip.
Solution: use something like
Math.max(STANDARD_WIDTH, innerLabel.contentWidth)

Tags: slider

Related branches

Revision history for this message
Roman Shchekin (mrqtros) wrote :
Roman Shchekin (mrqtros)
Changed in ubuntu-rssreader-app:
importance: Undecided → Medium
Revision history for this message
Roman Shchekin (mrqtros) wrote :

Any news?

Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
status: New → Triaged
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
status: Triaged → Confirmed
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
importance: Undecided → Medium
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: Undecided → Medium
status: New → Confirmed
Zoltan Balogh (bzoltan)
no longer affects: ubuntu-ui-toolkit
Revision history for this message
Michal Predotka (mpredotka) wrote :

Wanted to report a bug about this very problem but I see it's already reported. I'm attaching my screenshot anyway.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
assignee: nobody → Nekhelesh Ramananthan (nik90)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.2.1458+15.04.20150327-0ubuntu1

---------------
ubuntu-ui-toolkit (1.2.1458+15.04.20150327-0ubuntu1) vivid; urgency=medium

  [ By Tim Peeters ]
  * Remove the deprecated toolbar example from the gallery.
    Fixes LP: #1430904

  [ Nekhelesh Ramananthan ]
  * Fixes the slider bubble width to be dynamic and dependent
    on the label width. Fixes LP: #1228575

  [ Leo Arias ]
  * Added a scenario for autopilot tests without live dragging.
    Fixes LP: #1435698
  * Added a helper to reorder items in a list view.
    Fixes LP: #1435700

  [ Christian Dywan ]
  * Add tests/autopilot to sys.path of sphinx config.
  * Build-depend on python-autopilot used by sphinx-build
    Fixes LP: #1431749

  [ Zsombor Egri ]
  * Fixing crashes when AlarmModel::get() returned valid stock alarm
    ends up to be NULL in QML. Fixes LP: #1429273

  [ CI Train Bot ]
  * New rebuild forced.
 -- CI Train Bot <email address hidden> Fri, 27 Mar 2015 12:35:55 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Released
Roman Shchekin (mrqtros)
Changed in ubuntu-rssreader-app:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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