Opened 8 years ago

Last modified 2 weeks ago

#6987 reopened bug

InfoWindow Owner clipped in layout

Reported by: Karvjorm Owned by: aldeck
Priority: normal Milestone: R1
Component: Applications/Tracker Version: R1/Development
Keywords: Owner layout Cc: Karvjorm
Blocked By: Blocking: #14870
Has a Patch: no Platform: All

Description

As shown in the enclosed picture, "Owner" is clipped when it is shown with a localized text that is a bit longer than orginal in InfoWindow.

I tried to write a patch to resolve this problem, but because my development environment is broken (can't svn update the whole svn tree [due to a VMware virtual machine environment and a USB stick disk for it, I suppose]), I was not able to test it.

I enclosing here also a typo patch and another missing localization patch.

Attachments (6)

Owner.png (13.0 KB) - added by Karvjorm 8 years ago.
A clipped and localized InfoWindow "Owner" text
InfoWindow.patch (972 bytes) - added by Karvjorm 8 years ago.
Trying to solve a text clipping problem in a localized text.
FilePermissionView.patch (3.9 KB) - added by Karvjorm 8 years ago.
Two missing localization texts
NodeWalker.patch (376 bytes) - added by Karvjorm 8 years ago.
Typo in comment
infowindow.png (55.6 KB) - added by diver 21 months ago.
user-group-other.png (3.7 KB) - added by humdinger 21 months ago.
3 icons

Download all attachments as: .zip

Change History (29)

Changed 8 years ago by Karvjorm

Attachment: Owner.png added

A clipped and localized InfoWindow "Owner" text

Changed 8 years ago by Karvjorm

Attachment: InfoWindow.patch added

Trying to solve a text clipping problem in a localized text.

Changed 8 years ago by Karvjorm

Attachment: FilePermissionView.patch added

Two missing localization texts

Changed 8 years ago by Karvjorm

Attachment: NodeWalker.patch added

Typo in comment

comment:1 Changed 8 years ago by Karvjorm

Sorry, I mixed the comments of the two first comments.

comment:2 Changed 8 years ago by anevilyak

Owner: changed from anevilyak to aldeck
Status: newassigned

comment:3 Changed 8 years ago by humdinger

Blocking: 7295 added

(In #7295) You're right. Closing as duplicate.

comment:3 Changed 8 years ago by humdinger

Blocking: 7295 removed

The localization in InfoWindow.patch includes spaces, which can trip up translations at HTA.
FilePermissionView.patch doesn't solve the issue. I applied it locally, but the labels are still truncated, the column labels are now just as wide as a checkbox. This is hrev40612.

comment:4 Changed 7 years ago by diver

Component: Kits/libtracker.soApplications/Tracker
Version: R1/alpha2R1/Development

comment:5 Changed 6 years ago by dsjonny

I think the best way would be the usage of the layout api, as was in the find window. And also the capacity string is looks like wrong. (Using 45710 nightly.)

comment:6 Changed 3 years ago by pulkomandy

Increasing the width of the columns will not look very nice for languages where the strings are quite long. I would suggest using 45° rotated text for the column headers instead?

comment:7 Changed 21 months ago by pulkomandy

Resolution: fixed
Status: assignedclosed

Fixed in hrev51189.

comment:8 Changed 21 months ago by diver

Resolution: fixed
Status: closedreopened

"Execution" is still cut off in Russian locale. As well as the first latter of rotated "Others" label.

Changed 21 months ago by diver

Attachment: infowindow.png added

comment:9 Changed 21 months ago by diver

Has a Patch: set

comment:10 Changed 21 months ago by diver

comment:11 Changed 21 months ago by diver

Has a Patch: unset

comment:12 Changed 21 months ago by axeld

I'm not too fond of the rotated text; I'd rather see wider columns.

Other options:

  • U G O as headers, with a legend (user, group, others).
  • 1. 2. 3. as headers, with a legend.
  • icons as headers, with a legend.
  • Put header in two rows (alternating).

comment:13 Changed 21 months ago by diver

BTW, Alt+p (to expand permission view) doesn't seem to work anymore.

comment:14 in reply to:  12 Changed 21 months ago by dsjonny

Replying to axeld:

I'm not too fond of the rotated text; I'd rather see wider columns.

Other options:

  • U G O as headers, with a legend (user, group, others).
  • 1. 2. 3. as headers, with a legend.
  • icons as headers, with a legend.
  • Put header in two rows (alternating).

Hi

I "vote" for icons.

comment:15 in reply to:  12 ; Changed 21 months ago by humdinger

Replying to axeld:

  • icons as headers, with a legend.

Like that idea best. Instead of a legend, how about tool tips for the icons?
I'd suggest the "x-person" filetype icon for "owner" and the "People" icon for "group". What to use for the "other" though?

comment:16 in reply to:  13 ; Changed 21 months ago by humdinger

Replying to diver:

BTW, Alt+p (to expand permission view) doesn't seem to work anymore.

Works here in hrev51192 (gcc2h), fwiw.

comment:17 in reply to:  16 Changed 21 months ago by diver

Replying to humdinger:

Works here in hrev51192 (gcc2h), fwiw.

Right, I was using it was an issue with VMware Fusion keyboard mapping. Works now! :)

comment:18 in reply to:  15 Changed 21 months ago by humdinger

Replying to humdinger:

I'd suggest the "x-person" filetype icon for "owner" and the "People" icon for "group". What to use for the "other" though?

Here's the above and a new ???-icon for the "other". These are 24px, don't look that nice in 16px...

3 icons

Changed 21 months ago by humdinger

Attachment: user-group-other.png added

3 icons

comment:19 Changed 21 months ago by humdinger

Has a Patch: set

comment:20 Changed 21 months ago by humdinger

Has a Patch: unset

comment:21 Changed 2 weeks ago by diver

Blocking: 14870 added

comment:22 Changed 2 weeks ago by -Meanwhile-

Sorry for the duplicate 14870...

'Put header in two rows (alternating)' seems like the second best solution to me (after the rotated text).

Last edited 2 weeks ago by -Meanwhile- (previous) (diff)
Note: See TracTickets for help on using tickets.