Changeset View
Changeset View
Standalone View
Standalone View
src/panels/information/informationpanel.cpp
| Show First 20 Lines • Show All 170 Lines • ▼ Show 20 Line(s) | |||||
| 171 | void InformationPanel::showContextMenu(const QPoint &pos) { | 171 | void InformationPanel::showContextMenu(const QPoint &pos) { | ||
| 172 | QMenu popup(this); | 172 | QMenu popup(this); | ||
| 173 | 173 | | |||
| 174 | QAction* previewAction = popup.addAction(i18nc("@action:inmenu", "Preview")); | 174 | QAction* previewAction = popup.addAction(i18nc("@action:inmenu", "Preview")); | ||
| 175 | previewAction->setIcon(QIcon::fromTheme(QStringLiteral("view-preview"))); | 175 | previewAction->setIcon(QIcon::fromTheme(QStringLiteral("view-preview"))); | ||
| 176 | previewAction->setCheckable(true); | 176 | previewAction->setCheckable(true); | ||
| 177 | previewAction->setChecked(InformationPanelSettings::previewsShown()); | 177 | previewAction->setChecked(InformationPanelSettings::previewsShown()); | ||
| 178 | 178 | | |||
| 179 | QAction* previewAutoPlayAction = popup.addAction(i18nc("@action:inmenu", "Preview autoPaly")); | ||||
ngraham: Typo.
Also, I would re-word to say "Auto-play" | |||||
| 180 | previewAutoPlayAction->setIcon(QIcon::fromTheme(QStringLiteral("media-playback-start"))); | ||||
| 181 | previewAutoPlayAction->setCheckable(true); | ||||
| 182 | previewAutoPlayAction->setChecked(InformationPanelSettings::previewsAutoPlay()); | ||||
| 183 | | ||||
| 179 | QAction* configureAction = popup.addAction(i18nc("@action:inmenu", "Configure...")); | 184 | QAction* configureAction = popup.addAction(i18nc("@action:inmenu", "Configure...")); | ||
| 180 | configureAction->setIcon(QIcon::fromTheme(QStringLiteral("configure"))); | 185 | configureAction->setIcon(QIcon::fromTheme(QStringLiteral("configure"))); | ||
| 181 | 186 | | |||
| 182 | QAction* dateformatAction = popup.addAction(i18nc("@action:inmenu", "Condensed Date")); | 187 | QAction* dateformatAction = popup.addAction(i18nc("@action:inmenu", "Condensed Date")); | ||
| 183 | dateformatAction->setIcon(QIcon::fromTheme(QStringLiteral("change-date-symbolic"))); | 188 | dateformatAction->setIcon(QIcon::fromTheme(QStringLiteral("change-date-symbolic"))); | ||
| 184 | dateformatAction->setCheckable(true); | 189 | dateformatAction->setCheckable(true); | ||
| 185 | dateformatAction->setChecked(InformationPanelSettings::dateFormat() == static_cast<int>(Baloo::DateFormats::ShortFormat)); | 190 | dateformatAction->setChecked(InformationPanelSettings::dateFormat() == static_cast<int>(Baloo::DateFormats::ShortFormat)); | ||
| 186 | 191 | | |||
| Show All 17 Lines | 208 | } else if (action == configureAction) { | |||
| 204 | FileMetaDataConfigurationDialog* dialog = new FileMetaDataConfigurationDialog(this); | 209 | FileMetaDataConfigurationDialog* dialog = new FileMetaDataConfigurationDialog(this); | ||
| 205 | dialog->setDescription(i18nc("@label::textbox", | 210 | dialog->setDescription(i18nc("@label::textbox", | ||
| 206 | "Select which data should be shown in the information panel:")); | 211 | "Select which data should be shown in the information panel:")); | ||
| 207 | dialog->setItems(m_content->items()); | 212 | dialog->setItems(m_content->items()); | ||
| 208 | dialog->setAttribute(Qt::WA_DeleteOnClose); | 213 | dialog->setAttribute(Qt::WA_DeleteOnClose); | ||
| 209 | dialog->show(); | 214 | dialog->show(); | ||
| 210 | connect(dialog, &FileMetaDataConfigurationDialog::destroyed, m_content, &InformationPanelContent::refreshMetaData); | 215 | connect(dialog, &FileMetaDataConfigurationDialog::destroyed, m_content, &InformationPanelContent::refreshMetaData); | ||
| 211 | } | 216 | } | ||
| 212 | if (action == dateformatAction) { | 217 | else if (action == dateformatAction) { | ||
| 213 | int dateFormat = static_cast<int>(isChecked ? Baloo::DateFormats::ShortFormat : Baloo::DateFormats::LongFormat); | 218 | int dateFormat = static_cast<int>(isChecked ? Baloo::DateFormats::ShortFormat : Baloo::DateFormats::LongFormat); | ||
| 214 | 219 | | |||
| 215 | InformationPanelSettings::setDateFormat(dateFormat); | 220 | InformationPanelSettings::setDateFormat(dateFormat); | ||
| 216 | m_content->refreshMetaData(); | 221 | m_content->refreshMetaData(); | ||
| 217 | } | 222 | } | ||
| 223 | else if (action == previewAutoPlayAction) { | ||||
| 224 | InformationPanelSettings::setPreviewsAutoPlay(isChecked); | ||||
| 225 | m_content->setPreviewAutoPlay(isChecked); | ||||
| 226 | } | ||||
| 218 | } | 227 | } | ||
elvisangelaccio: Please move the `else`s at the end of the previous lines. | |||||
| 219 | 228 | | |||
| 220 | void InformationPanel::showItemInfo() | 229 | void InformationPanel::showItemInfo() | ||
| 221 | { | 230 | { | ||
| 222 | if (!isVisible()) { | 231 | if (!isVisible()) { | ||
| 223 | return; | 232 | return; | ||
| 224 | } | 233 | } | ||
| 225 | 234 | | |||
| 226 | cancelRequest(); | 235 | cancelRequest(); | ||
| ▲ Show 20 Lines • Show All 194 Lines • Show Last 20 Lines | |||||
Typo.
Also, I would re-word to say "Auto-play"