Robin S Posted November 17, 2017 Share Posted November 17, 2017 11 minutes ago, tpr said: I guess I can show the save overlay for a second and then remove. Or use a "saving" indicator that does not cover/lock the screen but just overlays a small part of it. Link to comment Share on other sites More sharing options...
Robin S Posted November 17, 2017 Share Posted November 17, 2017 Okay, I've narrowed down the problem. It occurs whenever an image or file is uploaded and then the page is immediately saved via the hotkey. It seems to be related to the "Choose File" button having focus after a image/file is uploaded - like the first click() just shifts the focus but does not trigger a submit. What fixed it for me was, at the end of aos_triggerSave() where the button click is triggered, first focus the button: aos_saveButton.focus().click(); 2 1 Link to comment Share on other sites More sharing options...
dotnetic Posted November 27, 2017 Share Posted November 27, 2017 Hi @tpr. I found a bug if you use AdminOnSteroids with the latest AdminThemeUIKit in ProcessWire 3.0.84. If you have activated the setting "Edit modules in modal on long-click" under Module tweaks, then the Save Button has a weird behavior. First of all it does not work anymore, and second it displays multiple buttons (see screenshot). Another problem is, that the CTRL-S shortcut does not work with AdminThemeUIKit. It would be great if you could add this. Cheers Jens 1 Link to comment Share on other sites More sharing options...
gmclelland Posted November 27, 2017 Share Posted November 27, 2017 @jmartsch - the CTRL-S works for me with AdminThemeUIKit. Link to comment Share on other sites More sharing options...
Macrura Posted November 28, 2017 Share Posted November 28, 2017 For any folks needing IUC on a ProFields table URL field, i have a working javascript snippet Link to comment Share on other sites More sharing options...
tpr Posted November 28, 2017 Author Share Posted November 28, 2017 @jmartsch Are you sure it's the long-click module setting that causes that? I can't reproduce it using PW 3.0.84 and the uikit theme that comes with it. @Macrura Could you share? As I remember there were no CSS class on those inputs that I could use to distinguish URL fields. Link to comment Share on other sites More sharing options...
Macrura Posted November 28, 2017 Share Posted November 28, 2017 in this case i hardcoded the field name into this script; i guess it would be possible to have some setting in AOS where one could specify a name of the table field, and then the name of the url field – once you know those 2 things it should be possible to have this be more generic, but haven't gotten that far yet... JS: $(function(){ $('li.Inputfield_links_table tr').each(function(){ urlInput = $(this).find('input[name*="_link_href"]'); urlValue = urlInput.val(); // the url entered urlInput.parent().addClass('urlFieldContainer'); if(urlValue) { var link = '<a style="height: 33px; line-height: 33px;" data-iuc-loaded="1" '+ 'href="'+urlValue+'" data-iuc-mode="button" class="iuc iuc-button iuc-table-button button-right pw-blank" data-iuc-force-http="1" '+ 'target="_blank"><i class="fa fa-arrow-right"></i></a>'; urlInput.after(link); } }); }); CSS: .urlFieldContainer { position:relative; overflow: auto; height: 100%; } .iuc-table-button { right:5px; } 1 Link to comment Share on other sites More sharing options...
tpr Posted November 28, 2017 Author Share Posted November 28, 2017 Thanks, that could work as a workaround. I guess I would actually solve it differently but using the same idea (specifying fields manually). 1 Link to comment Share on other sites More sharing options...
bernhard Posted November 28, 2017 Share Posted November 28, 2017 1 hour ago, Macrura said: For any folks needing IUC on a ProFields table URL field, i have a working javascript snippet sorry but google didn't help, what is that IUC you are talking about? Link to comment Share on other sites More sharing options...
gmclelland Posted November 28, 2017 Share Posted November 28, 2017 Inputfield Url Checker - I had to look it up too 1 Link to comment Share on other sites More sharing options...
Macrura Posted November 28, 2017 Share Posted November 28, 2017 yeah, if you haven't used it yet, it almost becomes a 'can't live without' - on sites where you do use a URL field, it saves a ton of clicking, super convenient, been using it since way back when it was it's own module.. 2 Link to comment Share on other sites More sharing options...
tpr Posted November 28, 2017 Author Share Posted November 28, 2017 IUC in action: 3 Link to comment Share on other sites More sharing options...
dotnetic Posted November 29, 2017 Share Posted November 29, 2017 Regarding my post/bug mentioned here This only happens when I selected or sidebar (left). I know, it's experimental ^^ With the Traditional setting with masthead navigation everything works fine. Link to comment Share on other sites More sharing options...
tpr Posted November 29, 2017 Author Share Posted November 29, 2017 Thanks, now I have a slight clue what's going on Link to comment Share on other sites More sharing options...
bernhard Posted November 29, 2017 Share Posted November 29, 2017 thanks, i'm using IUC already and it's very handy - just didn't know the short name Link to comment Share on other sites More sharing options...
gmclelland Posted November 30, 2017 Share Posted November 30, 2017 Any one else having trouble with this module not detecting updates with Ryan's ProcessWireUpgrade module? Some of my sites are stuck at AOS version 1.6.71 and it but it also reports 1.6.7.2. As you can see in the picture, there is another version available but it doesn't give me an option to download/update it. Link to comment Share on other sites More sharing options...
tpr Posted November 30, 2017 Author Share Posted November 30, 2017 Sorry, that's my fault, I've mixed the string vs numeric versioning and that's why the chaos (adrian reminded me). Next time I'll double check 1 Link to comment Share on other sites More sharing options...
adrian Posted November 30, 2017 Share Posted November 30, 2017 Also, next time for string versions, I think you should leave off the trailing period Link to comment Share on other sites More sharing options...
tpr Posted December 1, 2017 Author Share Posted December 1, 2017 I've managed to solve the uikit issues mentioned by @jmartsch, will be available in the next update. I'm experimenting with a feature called "language indicators", which would come handy to see the active/inactive languages in the page list: The idea is to make it visible only if not all languages are active, otherwise list all languages (abbreviated to the first 3 chars of their titles). Inactive languages are marked with a line-through. I think it's the quite talkative at first sight, and doesn't mess up the page list if you have many languages. Thoughts? 1 Link to comment Share on other sites More sharing options...
szabesz Posted December 2, 2017 Share Posted December 2, 2017 6 hours ago, tpr said: make it visible only if not all languages are active, otherwise list all languages I'm not quite sure what you mean. Do you list all languages when they are not visible? btw, HUN crossed out (aka line-through) looked like Arabic to me. I had to zoom in see what it is. Link to comment Share on other sites More sharing options...
tpr Posted December 2, 2017 Author Share Posted December 2, 2017 If at least one language is inactive, then list all and strike through the inactive ones. My other idea is to show this language list in a tooltip somehow. 1 Link to comment Share on other sites More sharing options...
szabesz Posted December 2, 2017 Share Posted December 2, 2017 Maybe an indicator to show that there are inactive languages and when the user clicks it we get the tooltip? In a tooltip you have more room to use something else which can be better than a line-through. Link to comment Share on other sites More sharing options...
Macrura Posted December 4, 2017 Share Posted December 4, 2017 In developing the Field Descriptions Extended module, i looked at your code for how to hook into the Inputfield::render and then from there run the method to do my replacements on the field descriptions, and it's all good. But i got stuck at fields inside repeaters; I see that AOS doesn't add the edit link to those fields either; i'm guessing the hook doesn't run on those because they are rendered by ajax, so wondering how hard it will be to support the AOS field edit links for fields in repeaters, and similarly in my module.... Link to comment Share on other sites More sharing options...
Robin S Posted December 12, 2017 Share Posted December 12, 2017 Hi @tpr, AOS seems to be suppressing the "required" icon in the template editor. Without AOS: With AOS: 1 Link to comment Share on other sites More sharing options...
tpr Posted December 12, 2017 Author Share Posted December 12, 2017 Thanks, it was a .text() vs .html() issue, module updated 1.7.0. 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