Merge pull request kran27#7 from DashTheDev/main

Add Window Position Saving

Author: kran27

.gitignore

@@ -1,2 +1,5 @@
 .vs/
-out/
+.vscode/
+out/
+build/
+releases/

spotifyamp.cpp

@@ -369,6 +369,8 @@ bool MainWindow::Load() {
   InitThreading();
   RegisterForGlobalHotkeys();
 
+  LoadPosition(0, 0);
+
   bool compact = PrefReadBool(false, "compact");
   compact_ = !compact;
   SetCompact(compact);
@@ -845,8 +847,6 @@ void MainWindow::Paint() {
 
 void MainWindow::SetDoubleSize(bool v) {
   PlatformWindow::SetDoubleSize(v);
-  eq_window_->SetDoubleSize(v);
-  PrefWriteInt(v, "double_size");
   SetVisualizer(24, 43, 76, 16);
 }
 
@@ -980,7 +980,12 @@ void MainWindow::Perform(int cmd) {
     if (const char *s = PlatformReadClipboard())
       TspPlayerPlayContext(tsp_, s, NULL, 0);
     break;
-  case CMD_DOUBLE_SIZE: SetDoubleSize(!double_size()); break;
+  case CMD_DOUBLE_SIZE: { 
+    bool is_double_size(!double_size());
+    PrefWriteInt(is_double_size, "double_size");
+    SetDoubleSize(is_double_size); 
+    eq_window_->SetDoubleSize(is_double_size);
+    } break;
   case CMD_ALWAYS_ON_TOP: SetAlwaysOnTop(!always_on_top()); break;
   case CMD_COMPACT: SetCompact(!compact_); break;
   case CMD_LOGIN: ShowLoginDialog(); break;
@@ -1514,6 +1519,7 @@ void MainWindow::GlobalHotkey(int id) {
 // PLAYLIST WINDOW
 ///////////////////////////////////////////////////////////
 PlaylistWindow::PlaylistWindow(MainWindow *main_window) {
+  id_ = 1;
   main_window_ = main_window;
   hover_button_ = -1;
   Resize(WND_MAIN_W, WND_MAIN_H * 3);
@@ -1524,6 +1530,7 @@ PlaylistWindow::~PlaylistWindow() {
 }
 
 void PlaylistWindow::Load() {
+  LoadPosition(main_window.screen_rect()->left, main_window.screen_rect()->bottom);
   SetCompact(PrefReadBool(false, "pl.compact"));
   font_size_ = PrefReadInt(10, "pl.font_size");
   row_height_ = 13;
@@ -1997,6 +2004,7 @@ void PlaylistWindow::Perform(int cmd) {
 // EQUALIZER WINDOW
 ///////////////////////////////////////////////////////////
 EqWindow::EqWindow(MainWindow *main_window) {
+  id_ = 2;
   main_window_ = main_window;
   hover_button_ = -1;
   hover_eq_ = -1;
@@ -2014,6 +2022,8 @@ void EqWindow::Load() {
   gain_mode_ = PrefReadInt(0, "eq.gain_mode");
   CopyToTsp();
 
+  LoadPosition(main_window.screen_rect()->right, main_window.screen_rect()->top);
+  SetDoubleSize(PrefReadBool(false, "double_size"));
   SetCompact(PrefReadBool(false, "eq.compact"));
 }
 
@@ -2375,6 +2385,7 @@ void EqWindow::ShowPresets() {
 
 
 GenWindow::GenWindow() {
+  id_ = 3;
   hover_button_ = -1;
   left_button_down_ = false;
   Resize(275, 116*2+29);
@@ -2528,6 +2539,7 @@ void GenWindow::Paint() {
 ///////////////////////////////////////////////////////////
 
 CoverArtWindow::CoverArtWindow(MainWindow *main_window) {
+  id_ = 4;
   main_window_ = main_window;
   image_needs_load_ = false;
   bitmap_ = NULL;
@@ -2596,6 +2608,7 @@ void CoverArtWindow::SetImage(const char* image) {
 #include <vector>
 
 void CoverArtWindow::Load() {
+  LoadPosition(main_window.screen_rect()->right, main_window.screen_rect()->bottom);
 	if (image_ == "") { return; }
 
 	PlatformDeleteBitmap(bitmap_);
@@ -2680,12 +2693,9 @@ PlatformWindow *InitSpotamp(int argc, char **argv) {
   if (!main_window.Load())
     return NULL;
 
-  playlist_window.Move(main_window.screen_rect()->left, main_window.screen_rect()->bottom);
-  eq_window.Move(main_window.screen_rect()->right, main_window.screen_rect()->top);
-  coverart_window.Move(main_window.screen_rect()->right, main_window.screen_rect()->bottom);
-
   eq_window.Load();
   playlist_window.Load();
+  coverart_window.Load();
 
   InitVisualizer();
 

spotifyamp.h

@@ -294,6 +294,7 @@ class CoverArtWindow : public GenWindow {
 	explicit CoverArtWindow(MainWindow* main_window);
 	virtual ~CoverArtWindow();
 
+	void Load();
 	virtual void Paint();
 	virtual void OnClose();
 
@@ -302,7 +303,6 @@ class CoverArtWindow : public GenWindow {
 	void GotImagePart(TspImageDownloadResult* part);
 
 private:
-	void Load();
 
 	MainWindow* main_window_;
 	std::string image_;

window.cpp

@@ -174,6 +174,29 @@ void PlatformWindowBase<PlatformWindow>::Create(PlatformWindow *owner_window) {
 }
 
 
+template<>
+const char *PlatformWindowBase<PlatformWindow>::GetPositionKey(const char *side) {
+  static char leftKey[12]; 
+  snprintf(leftKey, sizeof(leftKey), "window_%d_%s", id_, side);
+  return leftKey;
+}
+
+
+template<>
+void PlatformWindowBase<PlatformWindow>::SavePosition() {
+  PrefWriteInt(screen_rect_.left, GetPositionKey("l"));
+  PrefWriteInt(screen_rect_.top, GetPositionKey("t"));
+}
+
+
+template<>
+void PlatformWindowBase<PlatformWindow>::LoadPosition(int def_left, int def_top) {
+  int left(PrefReadInt(def_left, GetPositionKey("l")));
+  int top(PrefReadInt(def_top, GetPositionKey("t")));
+  Move(left, top);
+}
+
+
 template<>
 void PlatformWindowBase<PlatformWindow>::SetDoubleSize(bool v) {
   if (v != double_size_) {

window.h

@@ -17,6 +17,9 @@ class PlatformWindowBase {
 
   void Create(T *owner_window);
 
+  void SavePosition();
+  void LoadPosition(int def_left, int def_top);
+
   // Double all pixels
   void SetDoubleSize(bool v);
   bool double_size() const { return double_size_; }
@@ -30,6 +33,8 @@ class PlatformWindowBase {
   void SetVisible(bool visible);
   bool visible() const { return visible_; }
 
+  virtual void Move(int l, int t) {}
+
   // Resize window. If you're in doublesize mode, these still return the normal sized size.
   void Resize(int w, int h);
   int width() const { return width_; }
@@ -48,6 +53,7 @@ class PlatformWindowBase {
   T *self() { return static_cast<T*>(this); }   
   Point HandleMouseDragOrSize(int dx, int dy, bool snap);
 
+  int id_;
   bool double_size_;
   bool always_on_top_;
   bool active_window_;
@@ -57,6 +63,8 @@ class PlatformWindowBase {
   int width_, height_;
   Rect screen_rect_;
   T *next_, *owner_;
+ private:
+  const char *GetPositionKey(const char *side);
 };
 
 #if defined(WITH_SDL)

window_win32.cpp

@@ -248,6 +248,7 @@ void PlatformWindow::MoveAllWindows() {
                       r->left, r->top,
                       r->right - r->left, r->bottom - r->top, SWP_NOZORDER | SWP_NOOWNERZORDER | SWP_NOACTIVATE);
       w->need_move_ = false;
+      w->SavePosition();
     }
   }
   EndDeferWindowPos(dp);