Fixing menu items, strings with a box and Capitalization
Add "@title:window" marker; format and identation fixes
Replace EXIF and exif occurrences to Exif
Details
- Reviewers
pino - Group Reviewers
KPhotoAlbum - Commits
- R463:184940183d75: l10n fixes
Diff Detail
- Lint
Lint Skipped - Unit
Unit Tests Skipped
I've added some suggestions inline. If @pino gives his ok I'm happy to accept this...
AnnotationDialog/Dialog.cpp | ||
---|---|---|
601 | This could be clearer written as "@title:window image %1 of %2 images" | |
Settings/ExifPage.cpp | ||
39 | Do the same capitalization rules as for EXIF apply for IPTC? (Although I have to say "Iptc" is barely readable compared to "IPTC"... | |
Viewer/ViewerWidget.cpp | ||
494 | I don't know if it's worth mentioning at this place, but the "detail" parameter is usually the zoom factor (e.g. "[ Zoom x1.3]") as set in Viewer::ImageDisplay::updateZoomCaption()... |
I stopped at Viewer/ViewerWidget.cpp, but the main point is: please do not add whitespace/indentation changes to this patch. This patch is already big because of all the i18n changes, no need to make it way bigger by adding unrelated changes; they make it harder to review what was the actual topic of the commit.
AnnotationDialog/Dialog.cpp | ||
---|---|---|
241 | extra whitespace changes | |
342–349 | extra whitespace changes | |
819–824 | extra whitespace changes | |
AnnotationDialog/ImagePreviewWidget.cpp | ||
89 ↗ | (On Diff #40822) | extra whitespace changes |
322–325 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
AnnotationDialog/ListSelect.cpp | ||
727 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
AnnotationDialog/ResizableFrame.cpp | ||
487 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
518 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
526 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
BackgroundTaskManager/JobModel.cpp | ||
98–101 ↗ | (On Diff #40822) | extra whitespace changes |
Browser/AbstractCategoryModel.cpp | ||
46–48 ↗ | (On Diff #40822) | extra whitespace changes |
Browser/Breadcrumb.cpp | ||
36 ↗ | (On Diff #40822) | extra whitespace changes |
Browser/OverviewPage.cpp | ||
169 | extra whitespace changes | |
300 | extra whitespace changes | |
DB/ImageDate.cpp | ||
351–356 ↗ | (On Diff #40822) | extra whitespace changes |
DB/ImageSearchInfo.cpp | ||
270 ↗ | (On Diff #40822) | extra whitespace changes |
283 ↗ | (On Diff #40822) | extra whitespace changes |
DateBar/DateBarWidget.cpp | ||
100–108 ↗ | (On Diff #40822) | extra whitespace changes |
708 ↗ | (On Diff #40822) | extra whitespace changes |
Exif/InfoDialog.cpp | ||
41 | extra whitespace changes | |
Exif/ReReadDialog.cpp | ||
41 | extra whitespace changes | |
Exif/SearchDialog.cpp | ||
40 | extra whitespace changes | |
84 | extra whitespace changes | |
93 | extra whitespace changes | |
149 | extra whitespace changes | |
400 | extra whitespace changes | |
Exif/SearchInfo.cpp | ||
183–188 ↗ | (On Diff #40822) | extra whitespace changes |
HTMLGenerator/HTMLDialog.cpp | ||
63 | extra whitespace changes | |
125–130 | extra whitespace changes | |
569 | extra whitespace changes | |
ImportExport/Export.cpp | ||
83 | extra whitespace changes | |
ImportExport/ImportDialog.cpp | ||
149 ↗ | (On Diff #40822) | extra whitespace changes |
MainWindow/AutoStackImages.cpp | ||
49 | extra whitespace changes | |
83–93 | extra whitespace changes | |
MainWindow/DeleteDialog.cpp | ||
36 | extra whitespace changes | |
MainWindow/FeatureDialog.cpp | ||
42 | extra whitespace changes | |
223–224 | extra whitespace changes | |
MainWindow/InvalidDateFinder.cpp | ||
43 | extra whitespace changes | |
80 | extra whitespace changes | |
MainWindow/RunDialog.cpp | ||
78–80 ↗ | (On Diff #40822) | extra whitespace changes |
MainWindow/SearchBar.cpp | ||
32 ↗ | (On Diff #40822) | extra whitespace changes |
MainWindow/StatisticsDialog.cpp | ||
137 ↗ | (On Diff #40822) | extra whitespace changes |
178 ↗ | (On Diff #40822) | extra whitespace changes |
MainWindow/TokenEditor.cpp | ||
40 | extra whitespace changes | |
MainWindow/Window.cpp | ||
753 | extra whitespace changes | |
1290 | extra whitespace changes | |
1850–1854 | extra whitespace/indentation changes | |
Plugins/CategoryImageCollection.cpp | ||
32 ↗ | (On Diff #40822) | extra whitespace changes |
Plugins/ImageCollection.cpp | ||
48–61 ↗ | (On Diff #40822) | extra whitespace changes |
Settings/ExifPage.cpp | ||
39 | ||
Settings/FileVersionDetectionPage.cpp | ||
151 | extra whitespace changes | |
Settings/GeneralPage.cpp | ||
49–51 | extra whitespace changes | |
Settings/SettingsDialog.cpp | ||
103 | extra whitespace changes | |
Settings/TagGroupsPage.cpp | ||
49 ↗ | (On Diff #40822) | extra whitespace changes |
58 ↗ | (On Diff #40822) | extra whitespace changes |
69 ↗ | (On Diff #40822) | extra whitespace changes |
78 ↗ | (On Diff #40822) | extra whitespace changes; also, here the space between "group" and "..." should be removed |
258–260 ↗ | (On Diff #40822) | extra whitespace changes |
340–342 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
359–376 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
466–483 ↗ | (On Diff #40822) | extra whitespace/indentation changes |
500 ↗ | (On Diff #40822) | extra whitespace changes |
563–568 ↗ | (On Diff #40822) | extra whitespace changes |
746–748 ↗ | (On Diff #40822) | extra whitespace changes |
759–760 ↗ | (On Diff #40822) | extra whitespace changes |
814 ↗ | (On Diff #40822) | extra whitespace/indentation changes; |
Settings/ThumbnailsPage.cpp | ||
103 | extra whitespace changes | |
ThumbnailView/Delegate.cpp | ||
251 ↗ | (On Diff #40822) | extra whitespace changes |
Utilities/Util.cpp | ||
120–134 ↗ | (On Diff #40822) | extra whitespace changes |
Viewer/CategoryImageConfig.cpp | ||
47 ↗ | (On Diff #40822) | extra whitespace changes |
56–80 ↗ | (On Diff #40822) | extra whitespace changes |
Viewer/SpeedDisplay.cpp | ||
45–51 ↗ | (On Diff #40822) | extra whitespace changes |
71 ↗ | (On Diff #40822) | extra whitespace changes |
Viewer/ViewerWidget.cpp | ||
145–165 | extra whitespace changes | |
190–217 | extra whitespace changes |
@Abella: Do you know if/when you can continue with the patch? It touches quite many files and I don't want to make your work harder by changing too much in KPhotoAlbum while you work on it...
Fixing menu items, strings with a box and Capitalization
Add "@title:window" marker
Replace EXIF and exif occurrences to Exif