AddWorldCityButton not aligned with certain languages

Bug #1371229 reported by Stefano Verzegnassi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Clock App
Fix Released
Low
Stefano Verzegnassi

Bug Description

Some language (e.g. Italian) has different length for words "Add city", so this makes the horizontalCenter of AddWorldCityButton variable (see attached screenshot).

If confirmed, I already have a local branch on my PC with a fix for it (added anchors.horizontalCenterOffset equal to the half of the difference between paintedWidth of "City" label and paintedWidth of "Add" label).

Related branches

Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :
Changed in ubuntu-clock-app:
status: New → Confirmed
importance: Undecided → Low
milestone: none → rtm
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

I was wondering who would report this. I was about to fix it :-). Nice catch though. I wonder if instead of adding a horizontalCenterOffset, you remove the row and horizontal align the add button with the page and then anchor the labels to the left and right of the add button?

Changed in ubuntu-clock-app:
assignee: nobody → Stefano Verzegnassi (verzegnassi-stefano)
Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :

Thus labels wouldn't be children of AbstractButton and the clickable area would be reduced.
If we extend AbstractButton to clockPage's width, we can keep the labels inside AddWorldCityButton.qml. Users will be able to tap also the labels, and the code will be clearer.

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

yeah I meant something like,

Abstract Button {
   width: page.width
   height: button.height

  CircularButton {
     anchors.centerIn: parent
  }

  Label1 anchored to the left of circularbutton
  Label2 anchorred to the right of circularbutton
}

So they all remain within the AddWorldCityButton.qml file

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

@Stefano, hey we will be pushing a clock app update at the end of the day. Would you be able to provide a patch for this by then? It would be nice to have this bug fix in the update.

Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :

@Nekhelesh, I was at the central police station to take the passport. The bugfix was on the list of things I have to do the next hour.
The MP will come soon. :)

Changed in ubuntu-clock-app:
status: Confirmed → In Progress
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

Fix committed into lp:ubuntu-clock-app at revision 107, scheduled for release in ubuntu-clock-app, milestone rtm

Changed in ubuntu-clock-app:
status: In Progress → Fix Committed
Changed in ubuntu-clock-app:
status: Fix Committed → 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.