Skip to content

Added a Today Widget with an enable and disable button#33

Open
fretn wants to merge 2 commits into
thomasfinch:masterfrom
fretn:todaywidget
Open

Added a Today Widget with an enable and disable button#33
fretn wants to merge 2 commits into
thomasfinch:masterfrom
fretn:todaywidget

Conversation

@fretn

@fretn fretn commented Oct 15, 2015

Copy link
Copy Markdown

No description provided.

@ANGOmarcello

Copy link
Copy Markdown
Contributor

Hey that is nice. :)
It would be cool to check if it is time to enable according to schedule every time the widget is refreshed.

@fretn

fretn commented Oct 15, 2015

Copy link
Copy Markdown
Author

Problem is that it has to open the Main Application so it basicly leaves the widget panel

@fretn

fretn commented Oct 15, 2015

Copy link
Copy Markdown
Author

Unless its possible to change the colors from inside the widget

@ANGOmarcello

Copy link
Copy Markdown
Contributor

It could be possible to just call the gamma filter from within the widget

@ANGOmarcello

Copy link
Copy Markdown
Contributor

This could also help with reboot persistence as it would be reapplied once the notification view is opened

@ylor

ylor commented Oct 15, 2015

Copy link
Copy Markdown

Super happy that somebody started work on it, but I feel like a UISlider or maybe a -/+ buttons that can increase/decrease orangeness could be super useful.

@ANGOmarcello

Copy link
Copy Markdown
Contributor

If this is merged I would constribute to it

@fretn

fretn commented Oct 16, 2015

Copy link
Copy Markdown
Author

An uislider will not work, swiping in one direction means moving to the notifications view

@arthurhammer

Copy link
Copy Markdown

It looks like this commit is missing TodayViewController.h? It doesn't build at this point.

@fretn

fretn commented Oct 16, 2015

Copy link
Copy Markdown
Author

indeed, xcode failed on me, I'll fix it

@ANGOmarcello

Copy link
Copy Markdown
Contributor

An UI Slider would also be against the Human Interface Guidlines. It's not like we are forced to obey them but they have a solid reason to exist.

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.

4 participants