Merge pull request #1645 from hovancik/idea

Refactor

Author: hovancik

.github/chatmodes/Plan.chatmode.md

@@ -15,5 +15,5 @@ The plan consists of a Markdown document that describes the implementation plan,
 * Requirements: A list of requirements for the feature or refactoring task.
 * Implementation Steps: A detailed list of steps to implement the feature or refactoring task.
 * Testing: A list of tests that need to be implemented to verify the feature or refactoring task.
-* Comments in code: Only if neccessary, when the code is not self-explanatory.
+* Comments in code: Do not add comments.
 

app/preferences-renderer.js

@@ -21,6 +21,12 @@ window.onload = async (e) => {
     document.querySelector('#enablePostponeLong').closest('div').style.marginBottom = '56px'
   }
 
+  if (settings.hidePreferencesFileLocation) {
+    document.querySelectorAll('[data-preferences-file]').forEach(element => {
+      element.classList.add('hidden')
+    })
+  }
+
   new HtmlTranslate(document).translate()
   setWindowHeight()
   setTimeout(() => { eventsAttached = true }, 500)
@@ -77,7 +83,7 @@ window.onload = async (e) => {
         document.querySelector('#timeleft').innerHTML = timeleft
         document.querySelector('#breakNumber').innerHTML = breaknumber
         document.querySelector('#postponesNumber').innerHTML = postponesnumber
-        document.querySelector('#settingsfile').innerHTML = settings.hidePreferencesFileLocation ? 'hidden' : settingsfile
+        document.querySelector('#settingsfile').innerHTML = settingsfile
         document.querySelector('#logsfile').innerHTML = logsfile
         document.querySelector('#donotdisturb').innerHTML = doNotDisturb
         document.querySelector('#node').innerHTML = await window.process.node()

app/preferences.html

@@ -398,8 +398,10 @@
           postponesNumber: <span id="postponesNumber"></span>,
           Do Not Disturb: <span id="donotdisturb"></span>}
           <br />
+          <span data-preferences-file>
           settingsFile: <a href="" class="file" id="settingsfile"></a>
           <br />
+          </span>
           logsFile: <a href="" class="file" id="logsfile"></a>
           <br />Node.js: <span id="node"></span>,
           Chromium: <span id="chrome"></span>,