diff options
| author |  Martin Měřinský <mermar@centrum.cz> | 2017-11-29 10:57:08 +0100 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2017-11-30 23:14:46 -0800 |
| commit | 8ef87e618afd1ec943c0f42a852b3b393c1e2113 (patch) | |
| tree | c767de27b2a78d3a85575b2c15ddecd7edcca891 /desktop-widgets/locationinformation.cpp | |
| parent | 0855f6f31577789888aa6d9b955c1bc46ed34d40 (diff) | |
| download | subsurface-8ef87e618afd1ec943c0f42a852b3b393c1e2113.tar.gz | |
Use better aliases for icons.
Icon aliases were complete mess.
Some icons had alias some didn't.
Named with underscores vs. hyphens vs. camelCase.
Lower vs. upper case.
"ICON" prefix vs. suffix vs. nothing.
With vs. without filename suffix.
Some didn't make sence. Eg. mapwidget-marker-gray
(I can see, it's grey, but what does it represent?)
Some were duplicated, eg warning vs. warning-icon.
Some were name after widget, which is wrong.
Do not reinvent wheel. Use widely used naming scheme
close to Freedesktop Icon Naming Specification. This
will enable usage of common icons from current set in
the future. Thus Subsurface will fit nicely to GUI.
This changes icon aliases to one, easy grep-able style.
Signed-off-by: Martin Měřinský <mermar@centrum.cz>
Diffstat (limited to 'desktop-widgets/locationinformation.cpp')
| -rw-r--r-- | desktop-widgets/locationinformation.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/desktop-widgets/locationinformation.cpp b/desktop-widgets/locationinformation.cpp index 454c4c68f..5212b0bf9 100644 --- a/desktop-widgets/locationinformation.cpp +++ b/desktop-widgets/locationinformation.cpp @@ -376,8 +376,8 @@ void DiveLocationModel::resetModel() QVariant DiveLocationModel::data(const QModelIndex &index, int role) const { - static const QIcon plusIcon(":plus"); - static const QIcon geoCode(":geocode"); + static const QIcon plusIcon(":list-add-icon"); + static const QIcon geoCode(":geotag-icon"); if (index.row() <= 1) { // two special cases. if (index.column() == UUID) { |