[InformationPanel] Hide the video when the preview is disabled, avoid computing the...

[dolphin.git] / src / panels / information / informationpanel.cpp

diff --git a/src/panels/information/informationpanel.cpp b/src/panels/information/informationpanel.cpp
index a864cb00574987e331411e00defe8b4b5ba6ed10..e5257bc72556839db0e55fa3570a98ffaaf92302 100644 (file)
--- a/src/panels/information/informationpanel.cpp
+++ b/src/panels/information/informationpanel.cpp
@@ -185,7 +185,8 @@ void InformationPanel::showContextMenu(const QPoint &pos) {
     dateformatAction->setChecked(InformationPanelSettings::dateFormat() == static_cast<int>(Baloo::DateFormats::ShortFormat));
 
     popup.addSeparator();
     dateformatAction->setChecked(InformationPanelSettings::dateFormat() == static_cast<int>(Baloo::DateFormats::ShortFormat));
 
     popup.addSeparator();

-    foreach (QAction* action, customContextMenuActions()) {
+    const auto actions = customContextMenuActions();
+    for (QAction *action : actions) {

         popup.addAction(action);
     }
 
         popup.addAction(action);
     }
 


@@ -198,8 +199,8 @@ void InformationPanel::showContextMenu(const QPoint &pos) {
 
     const bool isChecked = action->isChecked();
     if (action == previewAction) {
 
     const bool isChecked = action->isChecked();
     if (action == previewAction) {

-        m_content->setPreviewVisible(isChecked);

         InformationPanelSettings::setPreviewsShown(isChecked);
         InformationPanelSettings::setPreviewsShown(isChecked);

+        m_content->refreshPreview();

     } else if (action == configureAction) {
         FileMetaDataConfigurationDialog* dialog = new FileMetaDataConfigurationDialog(this);
         dialog->setDescription(i18nc("@label::textbox",
     } else if (action == configureAction) {
         FileMetaDataConfigurationDialog* dialog = new FileMetaDataConfigurationDialog(this);
         dialog->setDescription(i18nc("@label::textbox",


@@ -311,7 +312,7 @@ void InformationPanel::slotFilesAdded(const QString& directory)
 
 void InformationPanel::slotFilesChanged(const QStringList& files)
 {
 
 void InformationPanel::slotFilesChanged(const QStringList& files)
 {

-    foreach (const QString& fileName, files) {
+    for (const QString& fileName : files) {

         if (m_shownUrl == QUrl::fromLocalFile(fileName)) {
             showItemInfo();
             break;
         if (m_shownUrl == QUrl::fromLocalFile(fileName)) {
             showItemInfo();
             break;


@@ -321,7 +322,7 @@ void InformationPanel::slotFilesChanged(const QStringList& files)
 
 void InformationPanel::slotFilesRemoved(const QStringList& files)
 {
 
 void InformationPanel::slotFilesRemoved(const QStringList& files)
 {

-    foreach (const QString& fileName, files) {
+    for (const QString& fileName : files) {

         if (m_shownUrl == QUrl::fromLocalFile(fileName)) {
             // the currently shown item has been removed, show
             // the parent directory as fallback
         if (m_shownUrl == QUrl::fromLocalFile(fileName)) {
             // the currently shown item has been removed, show
             // the parent directory as fallback