tpr Posted August 3, 2018 Author Share Posted August 3, 2018 I've updated to work with Batch Child Editor's title fields. 5 Link to comment Share on other sites More sharing options...
matjazp Posted August 5, 2018 Share Posted August 5, 2018 If I enable "Disable filename truncation for File fields", the download icon is missing. I also have enabled Asset download button and Move to first/last buttons). 1 Link to comment Share on other sites More sharing options...
tpr Posted August 9, 2018 Author Share Posted August 9, 2018 It was a z-index issue and should be fixed in v2.0.2, thanks for reporting. 2 Link to comment Share on other sites More sharing options...
Pete Posted September 18, 2018 Share Posted September 18, 2018 Hi, when applying CodeMirror in AOS, when viewing source the text is black on a dark background. Nothing I've tried seems to be able to override this (custom stylesheets etc). Any ideas? 1 Link to comment Share on other sites More sharing options...
szabesz Posted September 18, 2018 Share Posted September 18, 2018 31 minutes ago, Pete said: Any ideas? It happens if the CKEditor skin is set to default. Probably @tpr always sets it to LightWire that is why he has not yet noticed it, I guess.... Link to comment Share on other sites More sharing options...
Pete Posted September 18, 2018 Share Posted September 18, 2018 This also happens on LightWire actually when using the UIKit admin theme. The CSS class that overrides the colour is this: It's bizarre that I can't override it even when using a custom CKE CSS file and adding color: #fffff !important (I kinda thought !important would do the trick). Link to comment Share on other sites More sharing options...
szabesz Posted September 18, 2018 Share Posted September 18, 2018 I see. I'm still on Reno, and get white text when LightWire is being used. 1 Link to comment Share on other sites More sharing options...
Pete Posted September 18, 2018 Share Posted September 18, 2018 Thanks - that definitely narrows it down to Uikit then. One step closer ? 1 Link to comment Share on other sites More sharing options...
Pete Posted September 21, 2018 Share Posted September 21, 2018 There's a UIKit admin theme bug when you switch on AoS - see here: https://github.com/processwire/processwire-issues/issues/706 The bug doesn't exist when you switch AoS off so just needs some style tweaks somewhere in AoS but I didn't investigate further than that sorry. 1 Link to comment Share on other sites More sharing options...
dotnetic Posted October 9, 2018 Share Posted October 9, 2018 Hey @tpr there are settings in AoS for PageListTweaks: Always show pagelist actions and Always show extra actions. Could you make them work also in a ListerPro? On 9/21/2018 at 11:57 AM, Pete said: There's a UIKit admin theme bug when you switch on AoS - see here: https://github.com/processwire/processwire-issues/issues/706 The bug doesn't exist when you switch AoS off so just needs some style tweaks somewhere in AoS but I didn't investigate further than that sorry. I can confirm this bug. Link to comment Share on other sites More sharing options...
tpr Posted October 9, 2018 Author Share Posted October 9, 2018 I know about that bug, will check later. As for ListerPro, the problem is that I still don't own a copy of it so it's hard to fix. If you happen to make a PR, that would be great. Link to comment Share on other sites More sharing options...
tpr Posted October 10, 2018 Author Share Posted October 10, 2018 The overlapped header button and the CodeMirror black font color issues should be fixed in v2.0.4. As for the latter, if you set a light CodeMirror theme then you should manually override the font color with a custom CSS. I also made the CodeMirror editor text a bit larger and force word-wrapping. 1 Link to comment Share on other sites More sharing options...
MilenKo Posted October 26, 2018 Share Posted October 26, 2018 Hello there. I am not sure if the issue I experience is a bug of AOS or there is some weird setting I have, but I noticed that on a PW latest version with just AOS installed, when I select the Sticky header option and scroll down the page a bit, the submenu's are not showing until I scroll back up where they are dropped originally. Any ideas on how to have that fixed as it is not that convenient to have the sticky header if you are not seeing your menu and still need to scroll back up to make it appear? Link to comment Share on other sites More sharing options...
tpr Posted October 28, 2018 Author Share Posted October 28, 2018 @MilenKo This is because these dropdowns are positioned via JavaScript (inline CSS) and it happens when you hover on the menu. This makes this weird behaviour that I have noticed as well but never examined it. As a simple fix I made those dropdown menus fixed positioned too in v2.0.5. Link to comment Share on other sites More sharing options...
adrian Posted November 22, 2018 Share Posted November 22, 2018 Hey @tpr - no big deal, but the most recent version of PW has broken the layout slightly when using AOS's compact header option. Link to comment Share on other sites More sharing options...
tpr Posted November 23, 2018 Author Share Posted November 23, 2018 You mean the header height? As I see it's only a matter of adding min-height: 54px to height: 54px to the CSS. 1 Link to comment Share on other sites More sharing options...
adrian Posted November 23, 2018 Share Posted November 23, 2018 2 hours ago, tpr said: You mean the header height? As I see it's only a matter of adding min-height: 54px to height: 54px to the CSS. Yep, that seems to fix it. 1 Link to comment Share on other sites More sharing options...
adrian Posted December 19, 2018 Share Posted December 19, 2018 Hey @tpr - I've started seeing something weird in the page tree with hidden pages. They don't show until I mouse over the tree itself: Let me know if you can't reproduce and I'll try to narrow it down for you. Link to comment Share on other sites More sharing options...
dotnetic Posted December 19, 2018 Share Posted December 19, 2018 Hey everybody and @tpr maybe you want to vote for a pull request I made: Expose actual user and role/s through ProcessWire.config javascript object https://github.com/processwire/processwire/pull/127 This would allow for example AdminOnSteroids to activate some features only if a specific user-role or user is active. But it is not exclusive for AoS. I really don't know if you should vote for the pull request or the feature-request or if Ryan cares about the votes. Link to comment Share on other sites More sharing options...
dotnetic Posted December 19, 2018 Share Posted December 19, 2018 @tpr There is a bug with displaying roles. I remember there was a select, so you can select which roles benefit from a specific setting. But under what conditions do these selects appear? I have one ProcessWire installation with latest AoS and AdminThemeUikit, where the roles select should be available, but does not show up, although it is there. This seems to come from the asmSelect class. When I remove the class, the select appears. With the old Default AdminTheme there appears a small triangle. Link to comment Share on other sites More sharing options...
tpr Posted December 19, 2018 Author Share Posted December 19, 2018 @adrian You're right, I cannot duplicate ? I'm not PWing nowadays too much but so far I haven't had such issue with hidden pages. @jmartsch Thanks, I think I found why the roles select wasn't displayed to you. It uses AOS's "Add placeholder to asmSelect" (under AsmTweaks), so if you do not check that you won't see the ROLES label. Plus UIkit uses a background-image to show the arrows for the select boxes and AOS was overriding it with a white background color, that's why they were not visible to you. Try v2.0.6 where you can see at least the arrows even if AsmTweaks is disabled. 1 Link to comment Share on other sites More sharing options...
adrian Posted December 19, 2018 Share Posted December 19, 2018 38 minutes ago, tpr said: @adrian You're right, I cannot duplicate ? I'm not PWing nowadays too much but so far I haven't had such issue with hidden pages. I don't think I have seen it before either, but on the site in question, it is triggered by the "Show Page IDs" option. If I turn that off, it works as expected. Link to comment Share on other sites More sharing options...
dotnetic Posted December 20, 2018 Share Posted December 20, 2018 @tpr Thanks for the update, the arrows now show up, but I think the UI could be improved and is also erroneous if you assign multiple roles. What do you think about this? Link to comment Share on other sites More sharing options...
tpr Posted December 20, 2018 Author Share Posted December 20, 2018 I'm fine with the inline roles, there's an Uikit CSS rule that makes them not wrapping in multiple lines. I'll remove that, thanks for the report. Link to comment Share on other sites More sharing options...
bernhard Posted December 23, 2018 Share Posted December 23, 2018 Hi @tpr I've had a problem when using the skip trash feature of AOS with ProCache, here is Ryan's explanation: Quote A couple suggestions. Avoid skipping the trash if possible. The trash is there for a reason, and there's a reason the core doesn't have a skip trash option. Consider that a 3rd party advanced power user tool that can have side effects, as it does here. If you are deleting pages from the API (skipping trash) you might want to clear the entire document cache after you are done by going into the admin in Setup > ProCache Is this feature only enabled for super users or for everyone in the admin? Maybe it would be better to trash pages first and delete them thereafter (automatically) rather than skipping the trash? Here is the complete post: Quote When it comes to clearing caches, ProCache is designed for the admin workflow. In your case a 3rd party module added a "Skip Trash" option that goes outside the intended admin workflow, so that's not something that ProCache is going to know about. If a page is deleted, ProCache is going to wipe any caches present for that Page either way. But it's not efficient for ProCache to execute clearing behaviors affecting other pages on $pages->delete() calls because these are usually performed in bulk, like when emptying trash. Not to mention they can be recursive. So if ProCache is executing multi-page cache clearing rules on every delete() call there's a real potential for significant overhead increase and potentially interfere with or prevent bulk delete() operations. A couple suggestions. Avoid skipping the trash if possible. The trash is there for a reason, and there's a reason the core doesn't have a skip trash option. Consider that a 3rd party advanced power user tool that can have side effects, as it does here. If you are deleting pages from the API (skipping trash) you might want to clear the entire document cache after you are done by going into the admin in Setup > ProCache. Or if you want to do it from the API you can do $procache->clearAll(); If you wanted multi-page cache clearing behaviors to be executed on every $pages->delete() call, this is where things get a little dangerous as there can be side effects as described above. But if you wanted to experiment with it, it could be done with a hook. You would save or trash a page before deleting it using a hook like this: $pages->addHookBefore('deleteReady', function($event) { $page = $event->arguments(0); $page->save(); }); While I don't recommend this in general, if your case has really simple cache clearing behaviors that don't involve lots of pages, then it may be just fine. Are you aware of that? 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