German translation: password wrong text is too long and destroys layout

Bug #1312749 reported by Jonas Schwabe
46
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Unity
Fix Released
High
Jonas Schwabe
7.2
Fix Released
High
Unassigned
unity (Ubuntu)
Fix Released
High
Jonas Schwabe
Trusty
Fix Released
Undecided
Unassigned

Bug Description

When I try to log in with a wrong password, the string which informs me about the wrong password is too long (I will attach a "screenshot") it makes the password box enlarge, which makes the right end of it (with the circle, that is spinning while logging in) disappear.

[Test Case]
(1) Log in to Unity
(2) Set the local to German (Use the Language Support panel in System Settings)
(3) Bring up the lockscren (eg. using control-alt-L)
(4) Attempt to unlock but enter an incorrect password

[Regression Potential]
Rendering error message text in the password dialog has been modified. It is possible the text may break incorrectly or the box may be incorrectly sized.

Related branches

Revision history for this message
Jonas Schwabe (jonas-schwabe) wrote :
Revision history for this message
Jonas Schwabe (jonas-schwabe) wrote :

I made the StaticCairoText have a maximum width so the text is being displayed in two lines if necessary.

Stephen M. Webb (bregma)
Changed in unity:
importance: Undecided → High
milestone: none → 7.2.1
status: New → Triaged
Changed in unity (Ubuntu):
status: New → In Progress
importance: Undecided → High
Changed in unity:
status: Triaged → In Progress
assignee: nobody → Jonas Schwabe (jonas-schwabe)
Changed in unity (Ubuntu):
assignee: nobody → Jonas Schwabe (jonas-schwabe)
Revision history for this message
Brandon Schaefer (brandontschaefer) wrote :

Confirmed patch working just fine. Ill upload it to a branch and get it merged.

Thanks for the patch!

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "diff" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Stephen M. Webb (bregma)
description: updated
Stephen M. Webb (bregma)
Changed in unity:
milestone: 7.2.1 → 7.3.0
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity - 7.2.0+14.10.20140512.4-0ubuntu1

---------------
unity (7.2.0+14.10.20140512.4-0ubuntu1) utopic; urgency=low

  [ Stephen M. Webb ]
  * lockscreen/LockScreenAccelerators.h: added required header for
    std:vector definition
  * disabled precompiled headers on arm64 Ubuntu package builds (LP:
    #1317276)

  [ Brandon Schaefer ]
  * Set the max width, so our static text wraps. So we don't end up
    rendering text out of the view area. (LP: #1312749)

  [ Marco Trevisan (Treviño) ]
  * UnityScreen: toggle activation of all the unity compiz actions when
    locking the screen Also, make sure that the lockscreen views are
    treated as "always-on-front" windows by nux (and this applies to
    both visibility and events).
  * PanelService: emit an invalid EntryActivated signal if the menu
    hasn't actually been shown In addition, if the menu is not visible
    after the first popup call, try to show a menu without keyboard
    focus. This will make possible to see menus also when there's a key-
    grab (although they will be without keyboard support). Finally,
    reset the menu state, if that failed.
  * UnityScreen: always paint the lockscreen above, just add menu and
    onboard as exceptions Some code cleanup, factorizing similar code.
    (LP: #1313280)
  * UnityScreen: don't try to show Dash/Hud if the screen is grabbed
    Also move the dash opening out from LauncherController, and get rid
    of UBus as first initialization source, as it can only slow things
    down here. (LP: #741869)
  * PluginAdapter: make sure we don't try to call an invalid
    initiate/terminate callback function (LP: #1221673)
  * DebugDBusInterface: match properties if they are in the AP array
    form [<type>, <value>] (LP: #1307748)

  [ William Hua ]
  * Add proper support for modifier-only shortcuts on the lock screen.
    (LP: #1291461)

  [ Jinkyu Yi ]
  * Dash: Fix problem at checking blacklisted unicode range (LP:
    #1251193)

  [ Andrea Azzarone ]
  * Add a warning in the session dialog if other sessions are still
    open. (LP: #1281058)
 -- Ubuntu daily release <email address hidden> Mon, 12 May 2014 22:03:27 +0000

Changed in unity (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Chris Halse Rogers (raof) wrote : Please test proposed package

Hello Jonas, or anyone else affected,

Accepted unity into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/unity/7.2.1+14.04.20140513-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in unity (Ubuntu Trusty):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Jonas, or anyone else affected,

Accepted unity into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/unity/7.2.1+14.04.20140513-0ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in unity:
status: In Progress → Fix Committed
Revision history for this message
Christopher Townsend (townsend) wrote :

I confirm after following the steps to reproduce in the bug description, I can no longer reproduce this issue with Unity in trusty-proposed.

$ apt-cache policy unity
unity:
  Installed: 7.2.1+14.04.20140513-0ubuntu2
  Candidate: 7.2.1+14.04.20140513-0ubuntu2
  Version table:
 *** 7.2.1+14.04.20140513-0ubuntu2 0
        100 /var/lib/dpkg/status
     7.2.0+14.04.20140423-0ubuntu1.2 0
        500 http://us.archive.ubuntu.com/ubuntu/ trusty-updates/main amd64 Packages
        500 http://security.ubuntu.com/ubuntu/ trusty-security/main amd64 Packages
     7.2.0+14.04.20140416-0ubuntu1 0
        500 http://us.archive.ubuntu.com/ubuntu/ trusty/main amd64 Packages

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity - 7.2.1+14.04.20140513-0ubuntu2

---------------
unity (7.2.1+14.04.20140513-0ubuntu2) trusty; urgency=medium

  [ Brandon Schaefer ]
  * Set the max width, so our static text wraps. So we don't end up
    rendering text out of the view area. (LP: #1312749)

  [ Jnkyu Yi ]
  * fixed range of blacklisted Unicode characters (lp: #1251193)

  [ Marco Trevisan (Treviño) ]
  * UnityScreen: toggle activation of all the unity compiz actions when
    locking the screen Also, make sure that the lockscreen views are
    treated as "always-on-front" windows by nux (and this applies to
    both visibility and events).
  * PanelService: emit an invalid EntryActivated signal if the menu
    hasn't actually been shown In addition, if the menu is not visible
    after the first popup call, try to show a menu without keyboard
    focus. This will make possible to see menus also when there's a key-
    grab (although they will be without keyboard support). Finally,
    reset the menu state, if that failed.
  * PluginAdapter: make sure we don't try to call an invalid
    initiate/terminate callback function (LP: #1221673)
  * DebugDBusInterface: match properties if they are in the AP array
    form [<type>, <value>] (LP: #1307748)

  [ Stephen M. Webb ]
  * lockscreen/LockScreenAccelerators.h: added required header for
    std:vector definition (fixes FTBFS in Ubuntu)
  * disabled precompiled headers on arm64 Ubuntu package builds
    (lp: #1317276)
  * bump upstream release to 7.2.1

  [ William Hua ]
  * added proper support for modifier-only shortcuts on the lock screen
    (lp: #1291461)
 -- Ubuntu daily release <email address hidden> Tue, 13 May 2014 20:46:01 +0000

Changed in unity (Ubuntu Trusty):
status: Fix Committed → Fix Released
Revision history for this message
Adam Conrad (adconrad) wrote : Update Released

The verification of the Stable Release Update for unity has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

Stephen M. Webb (bregma)
Changed in unity:
status: Fix Committed → Fix Released
Revision history for this message
henk (henk1234) wrote :

This bug still seems to exist in the dutch translation (see screenshot)
$ unity --version
unity 7.2.2

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Patches

Remote bug watches

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