WIP Set autologin profile via the GUI - Printable Version +- Kodi Community Forum (https://forum.kodi.tv) +-- Forum: Development (https://forum.kodi.tv/forumdisplay.php?fid=32) +--- Forum: Kodi Application (https://forum.kodi.tv/forumdisplay.php?fid=93) +--- Thread: WIP Set autologin profile via the GUI (/showthread.php?tid=162069) |
Set autologin profile via the GUI - leechguy - 2013-04-10 Hi, This is a continuation of the discussion on github. To avoid spamming all the XBMC devs we continue here. Set autologin profile via the GUI For those who want to test, this is the current implementation that is available in the nightlies: "Most recent" will be replaced by "Last used profile" to make it more clear which profile will be used for automatically logging in. "Most recent" is regarded as not obvious what is meant by it. This is the current behaviour in Frodo when the login screen is disabled. In this window the profile to use for auto login is selected. This solution is regarded as clunky and we're looking for a better way. Also here, the "Most recent" will be replaced by "Last used profile". Here the login screen is enabled. The Auto login field is disabled (grayed out). da-anda Wrote:ah, now I get it. Right. Login screen ofc disables autologin - but I don't think it's very userfriendly this way as it's not very obvious (at least it wasn't to me). As you can only have either login screen or autologin I'd combine them somehow, but I don't have a good solution for it yet. Note: I found a bug which I will investigate as it hinders my testing. When you delete the active profile, XBMC says bye, bye. Not caused by the PR, but was already in apparently. Besides, a daytime job hinders as well I would actually suggest to remove the possibility to remove the active profile. Seems kind of odd that this is currently possible. RE: Set autologin profile via the GUI - leechguy - 2013-04-10 I have fixed the bug that crashes XBMC when the active profile is being deleted. The PR is open. The PR also updates the "Most recent" text to "Last used profile". Tomorrow I'll take a look at the clunky UI and the proposal from da-anda to make it all a bit more user friendly. RE: Set autologin profile via the GUI - leechguy - 2013-05-01 I'm back after two weeks of skiing and have picked this up again. I have implemented two possible solutions to replace the dialog for selecting the profile to use for auto login (see the second screenshot in the first posting of this topic) and issued a PR as a request for comments on these solutions. The PR contains two possible solutions:
The solution suggested by da-anda has not been implemented yet. The benefit of the two possible solutions in this PR is that it requires only minor modifications to skins whereas the solution suggested by da-anda might have a bigger impact on skins due to the fact that the whole 'Settings - Profiles' window changes. Any comments on whether this is a better solution than using the original design with the selection dialog, or other possible solutions are most welcome. RE: Set autologin profile via the GUI - thebearnecessit - 2013-06-11 i prefer the 2nd solution shown in the last screenshot. RE: Set autologin profile via the GUI - leechguy - 2013-06-11 Yes, I like that one myself the most as well. The 'press the button to cycle through all profiles' option in the PR was more something I came up with because the button was already there to open the dialogbox shown in the second screen shot of posting #1. However, I don't think such a 'cycling' button is used anywhere else within XBMC and hence it is less intuitive. I'm waiting for feedback from the team on how to go forward on this. Now that the June merge window is closed, I hope that someone can find some time to comment. If not, that's ok with me too. For me, the most important thing was the ability to be able to set a default profile. Support for changing this setting via the GUI I see as a bonus and is already in (second screenshot of posting #1). The second option in the PR is nicer in my opinion. I'm glad you like it too |