Bug 7133 - End existing session check box label can overflow parent frame border
Summary: End existing session check box label can overflow parent frame border
Status: CLOSED WORKSFORME
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Client (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.15.0
Assignee: Bugzilla mail exporter
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-21 12:59 CET by Henrik Andersson
Modified: 2022-02-23 10:59 CET (History)
2 users (show)

See Also:
Acceptance Criteria:


Attachments
Removes the frame around checkbox (2.39 KB, patch)
2018-03-21 15:00 CET, Henrik Andersson
Details
Screenshot of patch applied (51.19 KB, image/png)
2018-03-21 15:01 CET, Henrik Andersson
Details

Description Henrik Andersson cendio 2018-03-21 12:59:58 CET
Currently the checkbox is surrounded by a frame which steals some pixels of the available space to show text. There is also a tight pack of widgets which prevents extending the area for text as:

.------------------------------. .----------------. .------------.
| [ ] End existing session     | |  Control Panel | | Options... |
'------------------------------'

Currently following translation will overflow the frame;
 
 fr it nl tr

On simple solution is to remove the frame to gain some more space for text.
Comment 1 Henrik Andersson cendio 2018-03-21 15:00:44 CET
Created attachment 845 [details]
Removes the frame around checkbox
Comment 2 Henrik Andersson cendio 2018-03-21 15:01:57 CET
Created attachment 846 [details]
Screenshot of patch applied

The patch fixes all problems with the current translations, there is enough room for the longest string as shown in this picture.
Comment 3 Pierre Ossman cendio 2022-02-23 10:59:51 CET
Not sure what was going on when this bug was filed since there is no screenshot applied. This issue does not happen with the current client though. But I cannot find any commit that would explain the change.

Since we don't have any more details we can't really dig further in this, so closing.

Note You need to log in before you can comment on or make changes to this bug.