My favorites | Sign in
Project Home Downloads Issues Source
Checkout   Browse   Changes  
Revision: r971
Author:  SubJunk
Date:  Nov 5, 2011
Review scores:
+1 
 
Positive:
Patrick....@gmail.com
Neutral:
chocol...@cpan.org
 

Log message

Always allow "Restart HTTP Server" button (#959) + formatting fixes

Affected files

    expand all   collapse all
Modify/trunk/ps3mediaserver/net/pms/PMS.javadiff
Modify/trunk/ps3mediaserver/net/pms/configuration/PmsConfiguration.javadiff
Comment by project member chocol...@cpan.org, Nov 6, 2011
Hmm. I'd forgotten what I had in mind for this when I replied. I was thinking of e.g. having the restart button greyed out (but not disabled) by default and switching to colour when a restart was required. I like the affordance of the UI notification that the server *needs* to be restarted while still allowing users to do it at any time without having to faff about making/undoing an arbitrary change.
Comment by subjunk, Nov 6, 2011
Ah ok. Feel free to implement that :)
Comment by project member ph.waeber, Nov 10, 2011
This would be a possibility http://www.ps3mediaserver.org/forum/viewtopic.php?f=14&t=12565#p60023
What do you think?
Comment by project member chocol...@cpan.org, Nov 10, 2011
Looks good. Feel free to beat me to it :-)
Comment by subjunk, Nov 10, 2011
Yeah looks good

Project members, sign in to write a code review
Powered by Google Project Hosting