mumble icon indicating copy to clipboard operation
mumble copied to clipboard

Minimize / Close: [X] Remember this choice

Open jaggzh opened this issue 2 years ago • 11 comments

When closing the mumble window, an option to remember our choice for the future would be nice (or we can put it in prefs -- or is it already in prefs?) Screenshot_20211012_104846

jaggzh avatar Oct 12 '21 17:10 jaggzh

It seems there is currently indeed no setting for this.

Related: #4682

Krzmbrzl avatar Oct 13 '21 06:10 Krzmbrzl

Can I try to get on this? Never contributed to open source before but I'm familiar with C++ and wanted to get started with open source.

Umang375 avatar Jan 14 '22 15:01 Umang375

@Umang375 absolutely! If you need anything or have any questions about the code, feel free to ask me. I'll see if I can answer ^^

Krzmbrzl avatar Jan 14 '22 18:01 Krzmbrzl

what are the tools and frameworks that i should install ?

Umang375 avatar Jan 15 '22 15:01 Umang375

https://github.com/mumble-voip/mumble/tree/master/docs/dev/build-instructions

davidebeatrici avatar Jan 15 '22 18:01 davidebeatrici

https://github.com/mumble-voip/mumble/tree/master/docs/dev/build-instructions

is there a slack channel or something like that, for communication?

Umang375 avatar Jan 16 '22 06:01 Umang375

We are using Matrix for that purpose. Our development channel is available at https://matrix.to/#/#mumble-dev:matrix.org

The general Mumble channel is linked in the main README of this repo

Krzmbrzl avatar Jan 16 '22 10:01 Krzmbrzl

I want to work on this issue if possible, @Krzmbrzl do you have any tips as to where I can start?

juanrodl avatar Jan 28 '22 17:01 juanrodl

in theory @Umang375 wanted to work on this already. Not sure what the status of that is though :eyes:

Krzmbrzl avatar Jan 31 '22 07:01 Krzmbrzl

Why is so hart to at a Option for this?

Steam,Discord,Thunderbird, all this have a Option for Minimize(certainly at Thunderbird the option breaks the Maximized option), but in a Voice Tool like Mumble or Discord,its a reale Usefull Feature to minimze in try by Using X.

Stefan-comkmits avatar Feb 15 '22 15:02 Stefan-comkmits

It's not hard. But someone has to do it 🤷

Krzmbrzl avatar Feb 15 '22 15:02 Krzmbrzl

Hey, I am an almost graduated B. Sc. Computer Science student from Berlin, Germany, and would like to do my first OSS contribution. Since this request is still open and I assume nothing has been done yet, I would like to give it a try. I guess, I just have to put a checkbox within the related XML file and save a flag for remembering the setting somewhere.

piorecki-simon avatar Aug 11 '22 15:08 piorecki-simon

@Merkaber yeah afaik there has not been progress on this front, so you are free to give this a shot.

The currently used dialog does not use a Qt UI file (yet), but is instead a standard Qt MessageBox: https://github.com/mumble-voip/mumble/blob/db8b6a3310d8891e49b1158204e62417a6fff03d/src/mumble/MainWindow.cpp#L509-L527

I'm not sure if there is an easy way to add a checkbox to this dialog and wire that up with the functionality that we need. If not, I think we could also simply recreate that dialog in Qt Designer as a dedicated .ui file (and then also write the corresponding .cpp and .h files). Long-term that will probably be the most customizable solution.

Krzmbrzl avatar Aug 11 '22 15:08 Krzmbrzl

Since Qt 5.2 it seems to be really easy to add a checkbox as mentioned here. Or is there a need to customize such dialogs further in the future and therefore a new UI element should be added as you pointed out?

PS: I joined the matrix room for a question related to the git workflow.

piorecki-simon avatar Aug 11 '22 17:08 piorecki-simon

Since Qt 5.2 it seems to be really easy to add a checkbox as mentioned here. Or is there a need to customize such dialogs further in the future and therefore a new UI element should be added as you pointed out?

Ah, nice. In that case just go ahead and make use of that. Should we ever require more customizations, we can still create a dedicated dialog UI for it :+1:

Krzmbrzl avatar Aug 11 '22 17:08 Krzmbrzl