Skip to content

added config file & remember window position & gradual audio decrease#29

Merged
Aurailus merged 6 commits into
Aurailus:masterfrom
zWolfrost:master
Apr 15, 2024
Merged

added config file & remember window position & gradual audio decrease#29
Aurailus merged 6 commits into
Aurailus:masterfrom
zWolfrost:master

Conversation

@zWolfrost

@zWolfrost zWolfrost commented Mar 2, 2024

Copy link
Copy Markdown
Contributor

-added config file at "~/.config/myxer/myxer.conf"
-added "remember last window position" option

Every option will get saved & loaded from the config file.

I'm kinda new to rust, hope that this PR respects your coding practices etc. (i have tried my best to do that).
If it doesn't, feel free to point out what you don't like about it.

I also found a little bug where if the audio source is stopped, the volume bar will get stuck on its last peak (this bug was also present previously) so if i manage to fix it i will open another PR for that.

-added config file at "~/.config/myxer/myxer.conf"
-added remember last window position option
also removed unnecessary println
@zWolfrost

zWolfrost commented Mar 3, 2024

Copy link
Copy Markdown
Contributor Author

I have added a gradual decrease to frozen* audio to fix the "little bug" i was talking about earlier. Also, it looks cool in my opinion. As always, if you don't like these features or only like some of them please feel free to tell me.
Also, to get more into technical details, the "REPETITIONS" constant dictates for how much update cycles the audio can be frozen* without decreasing. I have set it to "3" but maybe 5 or so is more consistent. I'll leave the decision to you.

@zWolfrost zWolfrost changed the title added config file & remember window position added config file & remember window position & gradual audio decrease Mar 3, 2024
Also removed unnecessary println & added some comments.
This fixes a bug where changing the "split channels" option makes the meters peaks full until they are refreshed by disabling and enabling the "visualize peaks" option.
@zWolfrost

Copy link
Copy Markdown
Contributor Author

This last commit fixes a bug where changing the "split channels" option makes the meters peaks full until they are refreshed by disabling and enabling the "visualize peaks" option. This is the last commit. I really hope this PR is not too overwhelming

@Aurailus

Aurailus commented Mar 4, 2024

Copy link
Copy Markdown
Owner

Hi, thank you for taking your time to make this PR! The lack of settings saving and the peak visualizations not working when you change between split channels were both issues I was hoping to get to eventually, but unfortunately I've not had any time.

I'm happy to merge this is and publish a new release. Before I do though, you can add your name to the credits if you'd like. It's in /src/about.

@zWolfrost

Copy link
Copy Markdown
Contributor Author

Hi, i have added my name, thank you for letting me.

Closing the program apparently secretly moves it somewhere on the screen for a frame before it gets actually closed.
I fixed bug where the program, when the close button is pressed, would sometime update its last remembered position to the position i mentioned.
Bugged me out for a while.
@zWolfrost

Copy link
Copy Markdown
Contributor Author

Hi, sorry for bothering you, but a month has passed and this PR has not seen any updates. I know you are busy with work, so I'm just making sure you remember about it, I'm not in an hurry or anything. If you want, i could maintain this project as well. Hope I'm not being too pushy :)

@Aurailus

Aurailus commented Apr 1, 2024

Copy link
Copy Markdown
Owner

Hey, sorry about the delay. I'm taking 5 classes rn in uni, and I'm in the middle of finals period. I should have time to merge this in the next week or so. thanks for your patience.

@Aurailus Aurailus merged commit d75efa7 into Aurailus:master Apr 15, 2024
@Aurailus

Copy link
Copy Markdown
Owner

Alright, I finally had a moment to check in on this properly. I've pulled the changes and ran them and they seem excellent! I'm going to publish a new release with these changes. Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants