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.

Sign up for GitHub

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

Jump to bottom

*TIME* is not aligning correctly. #367

Open
ZaxonXP opened this issue Apr 1, 2021 · 3 comments
Open

*TIME* is not aligning correctly. #367

ZaxonXP opened this issue Apr 1, 2021 · 3 comments

Comments

@ZaxonXP
Copy link

ZaxonXP commented Apr 1, 2021 •
edited
Loading

When using custom layout option like this youtube-viewer --custom-layout=' *TIME* | *ID* | *PUBLISHED* | *TITLE*' the output is not aligned when the results contains shorter and longer videos:

  11:34 | 63-EeFrBIqs | 16 Feb 2021 | Ciamajda smerfuje przyszłość • Epizod • Smerfy
  01:07:44 | yg6xpDI4x3g | 14 Feb 2021 | 60 minut Smerfnych przygód • Kompilacja muzyki • Smerfy
  11:38 | ZSkpsAH4ZFw | 11 Feb 2021 | Mała czarownica • Epizod • Smerfy

Could you please fix it so it would always be aligned to the right using spaces if the time is shorter than 8 characters (example below)?

     11:34 | 63-EeFrBIqs | 16 Feb 2021 | Ciamajda smerfuje przyszłość • Epizod • Smerfy
  01:07:44 | yg6xpDI4x3g | 14 Feb 2021 | 60 minut Smerfnych przygód • Kompilacja muzyki • Smerfy
     11:38 | ZSkpsAH4ZFw | 11 Feb 2021 | Mała czarownica • Epizod • Smerfy

Regards,
Piotr
@ZaxonXP ZaxonXP changed the title *TITLE* is not aligning correctly. *TIME* is not aligning correctly. May 5, 2021
ZaxonXP added a commit to ZaxonXP/youtube-viewer that referenced this issue May 5, 2021
@ZaxonXP
Fix for the issue trizen#367
a74f1cc 
Fixed format_time sub (return time string with 3 space prefix if the time is shorter than 1 hour)
@ZaxonXP
Copy link
Author

ZaxonXP commented May 5, 2021

Created a fix #370

@ZaxonXP
Copy link
Author

ZaxonXP commented May 5, 2021

However this "fix" will mess up the Duration: line in the display. :\

@ZaxonXP
Copy link
Author

ZaxonXP commented May 5, 2021

Fixed duration issue with the pull requests #371 and #372

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ZaxonXP