Add "Aspect Ratio" and "Frame Rate" to additional video information columns
ClosedPublic

Authored by kochihwu on Mar 27 2019, 3:26 AM.

Details

Summary

Add options to show aspect ratio and frame rate.

BUG: 366524
FIXED-IN: 19.08.0

Test Plan

Under Control -> Additional Info -> Video, there should be options for aspect ratio and frame rate.

Diff Detail

Repository
R318 Dolphin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kochihwu created this revision.Mar 27 2019, 3:26 AM
Restricted Application added a subscriber: kfm-devel. · View Herald TranscriptMar 27 2019, 3:26 AM
kochihwu requested review of this revision.Mar 27 2019, 3:26 AM
kochihwu edited the test plan for this revision. (Show Details)Mar 27 2019, 3:37 AM
ngraham accepted this revision.Mar 27 2019, 6:33 PM
ngraham added subscribers: elvisangelaccio, ngraham.

Nice, this works great!

Master only, @elvisangelaccio?

This revision is now accepted and ready to land.Mar 27 2019, 6:33 PM
ngraham edited the summary of this revision. (Show Details)Mar 27 2019, 6:33 PM

Never mind, of course master only, since it has new translated strings. @kochihwu can you provide your full name and email address so we can land this patch with correct authorship information? Thanks!

Thanks! It's great to be able to contribute to KDE.

Kochih Wu
mecca831@gmail.com

Just a side note: The extraction of the aspect ratio is completely broken before kf 5.57 in baloo/kfilemetadata, so it will show wrong results unless the data is re-indexed once kf 5.57 is used

Just a side note: The extraction of the aspect ratio is completely broken before kf 5.57 in baloo/kfilemetadata, so it will show wrong results unless the data is re-indexed once kf 5.57 is used

Thanks for the heads-up! Seems like that issue is below Dolphin though so it should be handled elsewhere. Also this is targeted at master, which means by the time KDE Applications 19.08 is released, Frameworks 5.57 will have been in the while for months already.

ngraham retitled this revision from Add video parameters "Aspect Ratio" and "Frame Rate" to the list of Additional Information columns to Add "Aspect Ratio" and "Frame Rate" to additional video information columns.Mar 27 2019, 8:21 PM
This revision was automatically updated to reflect the committed changes.