Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ShowInTaskbar feature does not work #34

Open
microgynon opened this issue Sep 19, 2016 · 8 comments
Open

ShowInTaskbar feature does not work #34

microgynon opened this issue Sep 19, 2016 · 8 comments
Milestone

Comments

@microgynon
Copy link

microgynon commented Sep 19, 2016

Using the oneclick installer, ShowInTaskbar = false feature doesn't work. There's still a listing in the taskbar with a shaded "progress bar" which I find really annoying.

Changing it to ShowInTaskbar = true doesn't seem to make a difference either.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/38006815-showintaskbar-feature-does-not-work?utm_campaign=plugin&utm_content=tracker%2F2840281&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2840281&utm_medium=issues&utm_source=github).
@floatas
Copy link
Collaborator

floatas commented Sep 20, 2016

I suppose you mean "Minimize to tray" setting ?
What OS ?

shaded "progress bar"

Can you provide a bit more info ?
Using latest YAPA version on Windows 8.1 it works fine.

@microgynon
Copy link
Author

While YAPA is not minimized (ie; you can see the clock timer in the top right hand side of the screen), the icon for YAPA is visible on the taskbar.

I'd like to make it so that YAPA never show up on the taskbar. Tried ShowInTaskbar = false, and it doesn't seem to make a difference.

Using windows 10.

@floatas
Copy link
Collaborator

floatas commented Sep 20, 2016

Yes, now I remember. Long time ago someone requested this feature.
There is a setting named ShowInTaskbar with default value True. Only way to change it is by editing settings file with notepad. But now with click once installer it's a bit harder to find config file.
MainWindow.xaml.cs
this.ShowInTaskbar = Properties.Settings.Default.ShowInTaskbar;

and when minimized application will appear in taskbar

private void MainWindow_StateChanged(object sender, EventArgs e)
        {
            if (this.WindowState == WindowState.Minimized && MinimizeToTray == true && Properties.Settings.Default.ShowInTaskbar)
            {
                Hide();
                sysTrayIcon.Visible = true;
            }

            if (!Properties.Settings.Default.ShowInTaskbar)
            {
                if (this.WindowState == WindowState.Minimized)
                {
                    sysTrayIcon.Visible = true;
                }
                else
                {
                    this.ShowInTaskbar = Properties.Settings.Default.ShowInTaskbar;
                }
            }

        }

I will try to think of better way, maybe set this setting from startup arguments.

@floatas floatas added this to the 2.0.0 milestone Oct 21, 2016
@fghber
Copy link
Contributor

fghber commented Mar 18, 2017

Actually I can think of four scenarios:

  • only show in tray
  • show in taskbar and tray
  • minimized to tray (current behavior)
  • never minimize to tray

@floatas
Copy link
Collaborator

floatas commented Mar 24, 2017

All of these scenarios are implemented in YAPA2, with this nasty piece of code:
https://github.com/YetAnotherPomodoroApp/YAPA-2/blob/master/YAPA/Plugins/MinimizeToTray/MinimizeToTray.cs#L128-L141

@fghber
Copy link
Contributor

fghber commented Mar 24, 2017

Cool! Is YAPA2 ready now?

@floatas
Copy link
Collaborator

floatas commented Mar 24, 2017

Well, it works, at least on my machine. There are still things that I would like to improve, but probably there always be something. I will try to find some time this week and update its description with links to installer and all.

@floatas
Copy link
Collaborator

floatas commented Mar 29, 2017

Fix from YAPA 2 can be ported here, just need to add one more setting.

@floatas floatas mentioned this issue May 29, 2017
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

No branches or pull requests

3 participants