Shows the a preview of the configured filename template on the save settings page. I'm not sure about the placement of the label but the functionality should be there. This also again includes a small refactor to access the placeholder replacement from outside of the ExportManager.
BUG: 381175
Details
Diff Detail
- Repository
- R166 Spectacle
- Branch
- preview
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 9664 Build 9682: arc lint + arc unit
Overall nice idea
src/Gui/SettingsDialog/SaveOptionsPage.cpp | ||
---|---|---|
220 | Not sure about the explicit label "Preview of filename" |
src/Gui/SettingsDialog/SaveOptionsPage.cpp | ||
---|---|---|
220 | "Filename preview" Also, let's format the filename in a monospaced font. Here's how you'd do that: xi18nc("@info", "Preview of filename: <filename>%1.%2</filename>", filename, mSaveImageFormat->currentText().toLower()) |
Very nice !
I would suggest you to position the preview right below the filename field, above the placeholder section.
It would make it visually clearer what it is for and easier to use.
I'm thinking of dropping the "Filename" now and just having "Preview:". For comparison:
I would be in favor as well since the context is clear now the preview is near the pattern.
You could even make Preview: a left label in the formlayout, so everything would align nicely. :)
Looks good! Just one little thing...
src/Gui/SettingsDialog/SaveOptionsPage.h | ||
---|---|---|
59 | Indentation |
src/Gui/SettingsDialog/SaveOptionsPage.cpp | ||
---|---|---|
129 | One more thing I missed, sorry: you don't need to add a space after the colon in the string. The formLayout ensures that there's adequate space automatically (and keeps all the colons aligned vertically). |