tpr Posted September 3, 2016 Author Share Posted September 3, 2016 That makes sense, thanks. Anyways, adding a general role based permission to each submodule in AOS would solve this too. The implementation seems not that hard but first I need to figure out the UI for this. Link to comment Share on other sites More sharing options...
tpr Posted September 4, 2016 Author Share Posted September 4, 2016 Ok, I think I got it, with a little help of JavaScript. Now I just need to allocate some time to add this to all submodules and beat this through the whole module 3 Link to comment Share on other sites More sharing options...
Juergen Posted September 4, 2016 Share Posted September 4, 2016 Wow! This looks absolutely awesome and is exactly the way I like it. Link to comment Share on other sites More sharing options...
adrian Posted September 4, 2016 Share Posted September 4, 2016 6 hours ago, tpr said: Ok, I think I got it, with a little help of JavaScript. Now I just need to allocate some time to add this to all submodules and beat this through the whole module I gotta say, I am super-impressed with the effort you have put into the settings for this module - so user friendly and beautiful! I have actually been thinking about adding user-roles for some panel permissions in Tracy and having a nice, compact, and functional settings interface like this would be exactly what is needed. After that glowing compliment can I say one negative thing I do wonder about the greyed-out checkboxes - they look to me like they can't be checked (disabled), rather than just not checked. It even affects the module uninstall checkbox which actually uses this visual cue to let you know that it can't be uninstalled. 2 Link to comment Share on other sites More sharing options...
tpr Posted September 4, 2016 Author Share Posted September 4, 2016 Thanks! When the number of the settings go high it's better to apply some UI tweaks to make it more readable. Most of them are CSS only, others have a bit of Js but they would be usable without Js too. I'll remove the greyed out check boxes, lets see how it goes. 4 Link to comment Share on other sites More sharing options...
Juergen Posted September 5, 2016 Share Posted September 5, 2016 Here is another issue if the headline is very long. As you can see the distance between the save button and the headline is very small. If you add following CSS snippet you can increase the distance a little bit more. #head_button { margin-bottom: 0 !important; } Here is what i looks like after inserting the changes: Its not a big problem but it looks like much better. Best regards Link to comment Share on other sites More sharing options...
tpr Posted September 5, 2016 Author Share Posted September 5, 2016 Thanks, I'll see if this whether it break anything. Here is the final UI for the roles. The asm selector went to the upper right corner of the submodule, having it below the label as before occupied too much space (almost doubled the upper fieldset height). It's not entirely consistent in FF-Chrome-IE but differences are mainly cosmetic so I don't tweak it more. Unfortunately the AOS page loads a bit slower because JS needs to change parent of 20+ asmFields which I'm afraid I can't optimize, but it's not that bad. Restriction are not applied so far but I hope it's only a matter of removing submodules from the enabledSubmodules array. But I feel that there will be exceptions here and there, as always 2 Link to comment Share on other sites More sharing options...
bernhard Posted September 5, 2016 Share Posted September 5, 2016 i found one issue with ajax loaded fields. it breaks the UI totally - no input possible any more: Link to comment Share on other sites More sharing options...
tpr Posted September 5, 2016 Author Share Posted September 5, 2016 4 hours ago, bernhard said: i found one issue with ajax loaded fields. it breaks the UI totally - no input possible any more: How to reproduce this? I wasn't able to when setting a CKEditor field to be ajax-loaded. Plus have you tried it without AOS? If this happens to AOS enabled only, could you narrow it down to any submodule? I have a few updates under the hood in my dev version but I don't think any of them would have fixed this. 1 Link to comment Share on other sites More sharing options...
tpr Posted September 5, 2016 Author Share Posted September 5, 2016 For the long headlines I may add a truncate option, with the entire text visible on hover (simple title). This is not nice but it's better than having the whole title invisible when it's pushed downwards. The problem is that the fixed header has fixed height, and some other fixed elements are dependent on this height, eg. notices, tabs, etc. So truncate would be the easiest/safest way of solving this I guess. 2 Link to comment Share on other sites More sharing options...
adrian Posted September 6, 2016 Share Posted September 6, 2016 Minor column width / padding issue 1 Link to comment Share on other sites More sharing options...
tpr Posted September 6, 2016 Author Share Posted September 6, 2016 4 hours ago, adrian said: Minor column width / padding issue Seems like you're using AOS. Great Here is a preview of truncating long titles: 2 Link to comment Share on other sites More sharing options...
tpr Posted September 6, 2016 Author Share Posted September 6, 2016 v059 is available with the features and fixes mentioned above. The 'Submodules' section in the admin has also got a minor redesign. 4 Link to comment Share on other sites More sharing options...
tpr Posted September 7, 2016 Author Share Posted September 7, 2016 Just committed a design update for the Submodules sections, see the updated screens in the previous post. Link to comment Share on other sites More sharing options...
Juergen Posted September 7, 2016 Share Posted September 7, 2016 Roland this looks really great!!! 1 Link to comment Share on other sites More sharing options...
bernhard Posted September 7, 2016 Share Posted September 7, 2016 i'm using modals for page edits a lot and have this issue. i had to switch AOS off completely. don't know when this error occured the first time (pw 3 and latest aos) Link to comment Share on other sites More sharing options...
tpr Posted September 7, 2016 Author Share Posted September 7, 2016 You mean the line on the top (sorry, your bug reports are often hard to understate, a picture itself doesn't say too much). It's the breadcrumb, just noticed that the 'modal' class is added to the 'body' in the default theme. Will fix soon. Any update on the ajax-loaded field issue? Is there any js error in the console? Link to comment Share on other sites More sharing options...
bernhard Posted September 7, 2016 Share Posted September 7, 2016 sorry i'm very busy atm and thought that's enough info for you that's how it should look like: all the tabs are on top. in my other screenshot the grey bar on top is fixed and overlaps all other content. the tabs have some margin to the modal top. regarding the ajax issue: it's somewhere on my todolist if i find more time, sorry Link to comment Share on other sites More sharing options...
tpr Posted September 7, 2016 Author Share Posted September 7, 2016 I think it's fine now, just committed a fix to github. The Reno theme has the 'modal' class on the html while the default theme has on the 'body' and I was unaware of this until now. Now I fixed this issue only but maybe there will be other places where it needs to be checked. 2 Link to comment Share on other sites More sharing options...
bernhard Posted September 7, 2016 Share Posted September 7, 2016 thanks, it's fixed 1 Link to comment Share on other sites More sharing options...
tpr Posted September 7, 2016 Author Share Posted September 7, 2016 v060 is up with several improvements and fixes to enabling/disabling the module. 1 Link to comment Share on other sites More sharing options...
tpr Posted September 8, 2016 Author Share Posted September 8, 2016 As discussed on Github, AOS will probably have a submodule that will load CKEditor plugins automatically, so you don't have to manually configure each field. I use the following plugins by default: autogrow keystrokes justify Let me know if you have such "basic" plugins you would like to see here. Obviously complex plugins that need more configuration will be not included here. Plus I don't know whether AOS should mess with CKE toolbar buttons, or leave it to the user to enter buttons to the field's settings in the admin. 2 Link to comment Share on other sites More sharing options...
tpr Posted September 8, 2016 Author Share Posted September 8, 2016 While by at this, the Lightwire skin will also an option for this submodule. I will apply some default CKE configuration for plugins but if the user uses a custom config.js then these defaults will be overwritten. This submodule is going to be a great timesaver for me, at least 2 Link to comment Share on other sites More sharing options...
tpr Posted September 8, 2016 Author Share Posted September 8, 2016 Prototype up & working Will try it for a while and add it to the next release. 4 Link to comment Share on other sites More sharing options...
bernhard Posted September 8, 2016 Share Posted September 8, 2016 hi tpr thats a great idea and has always annoyed me to need so much effort for only enabling justify buttons... thanks a lot, looking forward to it 2 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now