BKSalman / ytdlp-gui

a very simple GUI for ytdlp written in Rust
GNU General Public License v3.0
275 stars 16 forks source link

UX improvements #15

Closed eclairevoyant closed 1 year ago

eclairevoyant commented 1 year ago
BKSalman commented 1 year ago

Hey, thanks for the suggestions!

I will try them out once I have some free time, and see if I like em

I kinda agree with you anyways, but this project needs a bit of updating and cleaning up, so will do that when I have some free time

(Also I didn't get notified about this issue, so, I'm sorry for the late reply)

eclairevoyant commented 1 year ago

No worries, thanks for considering it either way.

I didn't get notified about this issue

The Github settings are a bit convoluted, you have to be "watching" this repo and then there is a separate notification setting controlling whether you get emails and/or GH notifs for watched repos. I just ran into the same issue for one of my repos 😅

BKSalman commented 1 year ago

The Github settings are a bit convoluted, you have to be "watching" this repo and then there is a separate notification setting controlling whether you get emails and/or GH notifs for watched repos. I just ran into the same issue for one of my repos sweat_smile

ah, got it, thanks, I just enabled it in my settings now :p

regarding the issue, will start working on updating the project to latest iced version, and refactor small things, then will try out the suggested UX changes, will do a PR so you can track the progress whenever I start working on all of these

BKSalman commented 1 year ago

I think this was addressed in #16