Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Formatting Issue with Dimensions and Synchronization Labels #15610

Closed
Lukiluc29 opened this issue Jun 17, 2024 · 9 comments · Fixed by #15621
Closed

Bug: Formatting Issue with Dimensions and Synchronization Labels #15610

Lukiluc29 opened this issue Jun 17, 2024 · 9 comments · Fixed by #15621
Labels
bug Something isn't working 👀 good first issue Good issue for new contributors to get started with

Comments

@Lukiluc29
Copy link
Contributor

Lukiluc29 commented Jun 17, 2024

Description

I have noticed a formatting issue with the labels "dimensions" and "synchronization" when hovering over an image. The colon is directly attached to the word "dimensions" resulting in "dimensions:" instead of "dimensions :". The same problem occurs with the "synchronization" line. To fix this, the colons should be removed from the following lines of code in the file "src/Files.App/Data/Items/ListedItem.cs" on line 49-50

if (!string.IsNullOrWhiteSpace(DimensionsDisplay))
    tooltipBuilder.Append($"{Environment.NewLine}{"PropertyDimensions".GetLocalizedResource()}: {DimensionsDisplay}");
if (SyncStatusUI.LoadSyncStatus)
    tooltipBuilder.Append($"{Environment.NewLine}{"syncStatusColumn/Header".GetLocalizedResource()}: {syncStatusUI.SyncStatusString}");

and ensure that "dimensions" and "dimensions:", as well as "status" and "status:", are properly localized.

Steps To Reproduce

  1. Hover over an image file
  2. Hover over a file stored in the cloud

Requirements

Correct the code so that the labels are formatted properly, with a space between the word and the colon.

Files Version

3.4.14.0

Windows Version

10.0.22631.3737

Log File

Capture d'écran 2024-06-17 125651

@Lukiluc29 Lukiluc29 added the bug Something isn't working 👀 label Jun 17, 2024
@0x5bfa 0x5bfa self-assigned this Jun 17, 2024
@yaira2
Copy link
Member

yaira2 commented Jun 17, 2024

There are two approaches to take here, the first is to have separate strings with and without the colons. The other approach is to have a dedicated string for colon.

@0x5bfa
Copy link
Member

0x5bfa commented Jun 17, 2024

As we have done, let's have DimentionsWithColon

@yaira2
Copy link
Member

yaira2 commented Jun 17, 2024

Sounds good

@0x5bfa
Copy link
Member

0x5bfa commented Jun 17, 2024

@Lukiluc29 Do you like to work on this?

@Lukiluc29
Copy link
Contributor Author

@Lukiluc29 Do you like to work on this?

I'm sorry, but I am unable to address this issue as I do not have a workstation to work on it.

@Lukiluc29
Copy link
Contributor Author

If someone else is willing to take on this task, it would be greatly appreciated.

@0x5bfa
Copy link
Member

0x5bfa commented Jun 17, 2024

Sure! I'll address this issue and make sure this will be included in the next stable build!

@yaira2 yaira2 added the good first issue Good issue for new contributors to get started with label Jun 17, 2024
@yaira2
Copy link
Member

yaira2 commented Jun 17, 2024

@0x5bfa if you don't mind, perhaps we can let someone else take this as it can be a good starting issue for a beginner.

@0x5bfa
Copy link
Member

0x5bfa commented Jun 17, 2024

Good idea! Let's leave.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 👀 good first issue Good issue for new contributors to get started with
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants