.stylelintrc.json

@@ -5,9 +5,6 @@
   "extends": [
     "stylelint-config-standard-scss"
   ],
-  "ignoreFiles": [
-    "src/themes/**.scss"
-  ],
   "rules": {
     "prettier/prettier": true,
     "scss/at-extend-no-missing-placeholder": null,

CHANGELOG.md

@@ -4,22 +4,6 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [5.8.0]
-
-- Updated Electron to 28.1.1 (fixes [325](https://github.com/Mastermindzh/tidal-hifi/issues/325))
-- Updated dependencies to latest
-  - added theme files to stylelint ignore
-  - fixed other stylelint errors
-
-- Added functionality to favorite a song (fixes [#323](https://github.com/Mastermindzh/tidal-hifi/issues/323))
-  - Added a hotkey to favorite ("Add to collection") songs: Control+a
-  - Added the "favorite" field in the `mediaInfo` and the API `/current` endpoint
-  - Added an endpoint to toggle favoriting a song: `http://localhost:47836/favorite/toggle`
-
-- Fixed wrong "end time stamp" for currently playing song (fixes [#282](https://github.com/Mastermindzh/tidal-hifi/issues/282))
-  - Affected the API + all integrations
-  - As requested we also added toggle to sync the timestamps to Discord (default = true)
-
 ## [5.7.1]
 
 - Fixed mpris not being set up correctly due to capitalization of the instance name.

build/electron-builder.base.yml

@@ -1,7 +1,7 @@
 appId: com.rickvanlieshout.tidal-hifi
-electronVersion: 28.1.1
+electronVersion: 24.1.2
 electronDownload:
-  version: 28.1.1+wvcus
+  version: 24.1.2+wvcus
   mirror: https://github.com/castlabs/electron-releases/releases/download/v
 snap:
   plugs:

package.json

@@ -1,13 +1,12 @@
 {
   "name": "tidal-hifi",
-  "version": "5.8.0",
+  "version": "5.7.1",
   "description": "Tidal on Electron with widevine(hifi) support",
   "main": "ts-dist/main.js",
   "scripts": {
     "start": "electron --inspect=0.0.0.0:5858 .",
     "watchStart": "nodemon dist -x \"npm run start\"",
     "compile": "tsc && npm run sass-and-copy",
-    "deps": "npm run watch",
     "watch": "tsc-watch --onSuccess \"npm run sass-and-copy\"",
     "copy-files": "copyfiles -u 1 --exclude './src/**/*.ts' --exclude './src/**/*.scss' \"./src/**/*\" ts-dist",
     "copy-themes-dev": "copyfiles -u 1 \"./themes/*\" node_modules/electron/dist/resources",
@@ -39,38 +38,38 @@
   "homepage": "https://github.com/Mastermindzh/tidal-hifi",
   "license": "MIT",
   "dependencies": {
-    "@electron/remote": "^2.1.1",
+    "@electron/remote": "^2.1.0",
-    "axios": "^1.6.5",
+    "axios": "^1.6.4",
     "discord-rpc": "^4.0.1",
     "electron-store": "^8.1.0",
     "express": "^4.18.2",
-    "hotkeys-js": "^3.13.3",
+    "hotkeys-js": "^3.12.1",
     "mpris-service": "^2.1.2",
     "request": "^2.88.2",
-    "sass": "^1.69.7"
+    "sass": "^1.69.5"
   },
   "devDependencies": {
     "@mastermindzh/prettier-config": "^1.0.0",
-    "@types/discord-rpc": "^4.0.8",
+    "@types/discord-rpc": "^4.0.5",
-    "@types/express": "^4.17.21",
+    "@types/express": "^4.17.17",
-    "@types/node": "^20.10.6",
+    "@types/node": "^20.4.4",
-    "@types/request": "^2.48.12",
+    "@types/request": "^2.48.8",
-    "@typescript-eslint/eslint-plugin": "^6.18.0",
+    "@typescript-eslint/eslint-plugin": "^6.1.0",
-    "@typescript-eslint/parser": "^6.18.0",
+    "@typescript-eslint/parser": "^6.1.0",
     "copyfiles": "^2.4.1",
-    "electron": "git+https://github.com/castlabs/electron-releases#v28.1.1+wvcus",
+    "electron": "git+https://github.com/castlabs/electron-releases.git#v24.1.2+wvcus",
-    "electron-builder": "^24.9.1",
+    "electron-builder": "^24.4.0",
-    "eslint": "^8.56.0",
+    "eslint": "^8.45.0",
     "js-yaml": "^4.1.0",
     "markdown-toc": "^1.2.0",
-    "nodemon": "^3.0.2",
+    "nodemon": "^3.0.1",
-    "prettier": "^3.1.1",
+    "prettier": "^3.0.0",
-    "stylelint": "^16.1.0",
+    "stylelint": "^15.10.2",
-    "stylelint-config-standard": "^36.0.0",
+    "stylelint-config-standard": "^34.0.0",
-    "stylelint-config-standard-scss": "^13.0.0",
+    "stylelint-config-standard-scss": "^10.0.0",
-    "stylelint-prettier": "^5.0.0",
+    "stylelint-prettier": "^4.0.0",
     "tsc-watch": "^6.0.4",
-    "typescript": "^5.3.3"
+    "typescript": "^5.1.6"
   },
   "prettier": "@mastermindzh/prettier-config"
-}
+}

scripts/verifyElements.js

@@ -16,12 +16,11 @@
     search: '[class^="searchField"]',
     shuffle: '*[data-test="shuffle"]',
     repeat: '*[data-test="repeat"]',
-    account: '*[class^="profileOptions"]',
+    account: '*[data-test^="profile-image-button"]',
-    settings: '*[data-test^="open-settings"]',
     media: '*[data-test="current-media-imagery"]',
     image: "img",
     current: '*[data-test="current-time"]',
-    duration: '*[class^=playbackControlsContainer] *[data-test="duration"]',
+    duration: '*[data-test="duration"]',
     bar: '*[data-test="progress-bar"]',
     footer: "#footerPlayer",
     mediaItem: "[data-type='mediaItem']",
@@ -30,7 +29,6 @@
     album_name_cell: '[class^="album"]',
     tracklist_row: '[data-test="tracklist-row"]',
     volume: '*[data-test="volume"]',
-    favorite: '*[data-test="footer-favorite-button"]',
   };
 
   let results = [];

src/constants/globalEvents.ts

@@ -12,5 +12,4 @@ export const globalEvents = {
   error: "error",
   whip: "whip",
   log: "log",
-  toggleFavorite: "toggleFavorite",
 };

src/constants/settings.ts

@@ -23,7 +23,6 @@ export const settings = {
   discord: {
     detailsPrefix: "discord.detailsPrefix",
     buttonText: "discord.buttonText",
-    includeTimestamps: "discord.includeTimestamps",
   },
   ListenBrainz: {
     root: "ListenBrainz",

src/models/mediaInfo.ts

@@ -10,5 +10,4 @@ export interface MediaInfo {
   current: string;
   duration: string;
   image: string;
-  favorite: boolean;
 }

src/models/options.ts

@@ -9,5 +9,4 @@ export interface Options {
   "app-name": string;
   image: string;
   icon: string;
-  favorite: boolean;
 }

src/pages/settings/preload.ts

@@ -48,7 +48,6 @@ let adBlock: HTMLInputElement,
   listenbrainz_delay: HTMLInputElement,
   enableWaylandSupport: HTMLInputElement,
   discord_details_prefix: HTMLInputElement,
-  discord_include_timestamps: HTMLInputElement,
   discord_button_text: HTMLInputElement;
 
 addCustomCss(app);
@@ -136,7 +135,6 @@ function refreshSettings() {
     ListenBrainzToken.value = settingsStore.get(settings.ListenBrainz.token);
     listenbrainz_delay.value = settingsStore.get(settings.ListenBrainz.delay);
     discord_details_prefix.value = settingsStore.get(settings.discord.detailsPrefix);
-    discord_include_timestamps.checked = settingsStore.get(settings.discord.includeTimestamps);
     discord_button_text.value = settingsStore.get(settings.discord.buttonText);
 
     // set state of all switches with additional settings
@@ -248,7 +246,6 @@ window.addEventListener("DOMContentLoaded", () => {
   ListenBrainzAPI = get("ListenBrainzAPI");
   ListenBrainzToken = get("ListenBrainzToken");
   discord_details_prefix = get("discord_details_prefix");
-  discord_include_timestamps = get("discord_include_timestamps");
   listenbrainz_delay = get("listenbrainz_delay");
   discord_button_text = get("discord_button_text");
 
@@ -283,6 +280,5 @@ window.addEventListener("DOMContentLoaded", () => {
   addInputListener(ListenBrainzToken, settings.ListenBrainz.token);
   addInputListener(listenbrainz_delay, settings.ListenBrainz.delay);
   addInputListener(discord_details_prefix, settings.discord.detailsPrefix);
-  addInputListener(discord_include_timestamps, settings.discord.includeTimestamps);
   addInputListener(discord_button_text, settings.discord.buttonText);
 });

src/pages/settings/settings.html

@@ -216,17 +216,6 @@
               </label>
             </div>
             <div id="discord_options">
-              <div class="group__option" class="hidden">
-                <div class="group__description">
-                  <h4>Include timestamps</h4>
-                  <p>Show current/end playtime in the Discord client</p>
-                </div>
-                <label class="switch">
-                  <input id="discord_include_timestamps" type="checkbox" />
-                  <span class="switch__slider"></span>
-                </label>
-              </div>
-
               <div class="group__option" class="hidden">
                 <div class="group__description">
                   <h4>Details prefix</h4>
@@ -400,7 +389,7 @@
           <img alt="tidal icon" class="about-section__icon" src="./icon.png" />
           <h4>TIDAL Hi-Fi</h4>
           <div class="about-section__version">
-            <a href="https://github.com/Mastermindzh/tidal-hifi/releases/tag/5.8.0">5.8.0</a>
+            <a href="">5.7.1</a>
           </div>
           <div class="about-section__links">
             <a href="https://github.com/mastermindzh/tidal-hifi/" class="about-section__button">Github <i
@@ -422,4 +411,4 @@
   </div>
 </body>
 
-</html>
+</html>

src/pages/settings/settings.scss

@@ -323,19 +323,16 @@ html {
     max-width: 500px;
     margin: -15px auto 0;
   }
-
   &__table {
     width: 120px;
-    margin: 0 auto;
+    margin: 0 auto 0;
 
     td {
       text-align: left;
     }
   }
-
   &__version {
-    margin: -10px 0 30px;
+    margin: -10px 0px 30px 0px;
-
     a {
       background-color: $tidal-grey-darker;
       border: none;
@@ -352,7 +349,6 @@ html {
   &__links {
     width: 300px;
     margin: 0 auto;
-
     a {
       border-radius: 10px;
       border: none;
@@ -422,11 +418,10 @@ html {
   align-items: center;
   width: 100%;
   max-width: 100%;
-  padding: 25px 0;
+  padding: 25px 0 25px 0px;
   border: 1px dashed $tidal-grey;
   border-radius: 3px;
   transition: 0.2s;
-
   &.is-active {
     background-color: $black;
   }
@@ -464,7 +459,6 @@ html {
   width: 100%;
   cursor: pointer;
   opacity: 0;
-
   &:focus {
     outline: none;
   }

src/preload.ts

@@ -45,7 +45,7 @@ const elements = {
   media: '*[data-test="current-media-imagery"]',
   image: "img",
   current: '*[data-test="current-time"]',
-  duration: '*[class^=playbackControlsContainer] *[data-test="duration"]',
+  duration: '*[data-test="duration"]',
   bar: '*[data-test="progress-bar"]',
   footer: "#footerPlayer",
   mediaItem: "[data-type='mediaItem']",
@@ -54,7 +54,6 @@ const elements = {
   album_name_cell: '[class^="album"]',
   tracklist_row: '[data-test="tracklist-row"]',
   volume: '*[data-test="volume"]',
-  favorite: '*[data-test="footer-favorite-button"]',
   /**
    * Get an element from the dom
    * @param {*} key key in elements object to fetch
@@ -132,10 +131,6 @@ const elements = {
     return this.get("volume").getAttribute("aria-checked") === "false"; // it's muted if aria-checked is false
   },
 
-  isFavorite: function () {
-    return this.get("favorite").getAttribute("aria-checked") === "true";
-  },
-
   /**
    * Shorthand function to get the text of a dom element
    * @param {*} key key in elements object to fetch
@@ -213,10 +208,6 @@ function addHotKeys() {
       handleLogout();
     });
 
-    addHotkey("Control+a", function () {
-      elements.click("favorite");
-    });
-
     addHotkey("Control+h", function () {
       elements.click("home");
     });
@@ -315,9 +306,6 @@ function addIPCEventListeners() {
         case globalEvents.pause:
           elements.click("pause");
           break;
-        case globalEvents.toggleFavorite:
-          elements.click("favorite");
-          break;
         default:
           break;
       }
@@ -528,7 +516,6 @@ setInterval(function () {
     "app-name": appName,
     image: "",
     icon: "",
-    favorite: elements.isFavorite(),
   };
 
   const titleOrArtistsChanged = currentSong !== songDashArtistTitle;

src/scripts/discord.ts

@@ -30,38 +30,30 @@ const observer = () => {
       settingsStore.get<string, string>(settings.discord.detailsPrefix) ?? "Listening to ";
     const buttonText =
       settingsStore.get<string, string>(settings.discord.buttonText) ?? "Play on TIDAL";
-    const includeTimestamps =
-      settingsStore.get<string, boolean>(settings.discord.includeTimestamps) ?? true;
-
-    let activity = {
-      ...idleStatus,
-      ...{
-        startTimestamp: includeTimestamps ? now : undefined,
-        endTimestamp: includeTimestamps ? remaining : undefined,
-      },
-    };
     if (mediaInfo.url) {
-      activity = {
+      rpc.setActivity({
-        ...activity,
+        ...idleStatus,
         ...{
           details: `${detailsPrefix}${mediaInfo.title}`,
           state: mediaInfo.artists ? mediaInfo.artists : "unknown artist(s)",
+          startTimestamp: now,
+          endTimestamp: remaining,
           largeImageKey: mediaInfo.image,
           largeImageText: mediaInfo.album ? mediaInfo.album : `${idleStatus.largeImageText}`,
           buttons: [{ label: buttonText, url: mediaInfo.url }],
         },
-      };
+      });
     } else {
-      activity = {
+      rpc.setActivity({
-        ...activity,
+        ...idleStatus,
         ...{
           details: `Watching ${mediaInfo.title}`,
           state: mediaInfo.artists,
+          startTimestamp: now,
+          endTimestamp: remaining,
         },
-      };
+      });
     }
-
-    rpc.setActivity(activity);
   }
 };
 

src/scripts/express.ts

@@ -40,9 +40,6 @@ export const startExpress = (mainWindow: BrowserWindow) => {
 
   if (settingsStore.get(settings.playBackControl)) {
     expressApp.get("/play", (req, res) => handleGlobalEvent(res, globalEvents.play));
-    expressApp.post("/favorite/toggle", (req, res) =>
-      handleGlobalEvent(res, globalEvents.toggleFavorite)
-    );
     expressApp.get("/pause", (req, res) => handleGlobalEvent(res, globalEvents.pause));
     expressApp.get("/next", (req, res) => handleGlobalEvent(res, globalEvents.next));
     expressApp.get("/previous", (req, res) => handleGlobalEvent(res, globalEvents.previous));

src/scripts/mediaInfo.ts

@@ -11,7 +11,6 @@ export const mediaInfo = {
   current: "",
   duration: "",
   image: "tidal-hifi-icon",
-  favorite: false,
 };
 
 export const updateMediaInfo = (arg: MediaInfo) => {
@@ -24,7 +23,6 @@ export const updateMediaInfo = (arg: MediaInfo) => {
   mediaInfo.current = propOrDefault(arg.current);
   mediaInfo.duration = propOrDefault(arg.duration);
   mediaInfo.image = propOrDefault(arg.image);
-  mediaInfo.favorite = arg.favorite;
 };
 
 /**

src/scripts/settings.ts

@@ -21,7 +21,6 @@ export const settingsStore = new Store({
     discord: {
       detailsPrefix: "Listening to ",
       buttonText: "Play on Tidal",
-      includeTimestamps: true,
     },
     ListenBrainz: {
       enabled: false,
@@ -62,13 +61,6 @@ export const settingsStore = new Store({
         migrationStore.get(settings.ListenBrainz.delay) ?? 5000
       );
     },
-    "5.8.0": (migrationStore) => {
-      console.log("running migrations for 5.8.0");
-      migrationStore.set(
-        settings.discord.includeTimestamps,
-        migrationStore.get(settings.discord.includeTimestamps) ?? true
-      );
-    },
   },
 });