tpr Posted August 3, 2018 Author Posted August 3, 2018 I've updated to work with Batch Child Editor's title fields. 5
matjazp Posted August 5, 2018 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
tpr Posted August 9, 2018 Author Posted August 9, 2018 It was a z-index issue and should be fixed in v2.0.2, thanks for reporting. 2
Pete Posted September 18, 2018 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
szabesz Posted September 18, 2018 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....
Pete Posted September 18, 2018 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).
szabesz Posted September 18, 2018 Posted September 18, 2018 I see. I'm still on Reno, and get white text when LightWire is being used. 1
Pete Posted September 18, 2018 Posted September 18, 2018 Thanks - that definitely narrows it down to Uikit then. One step closer ? 1
Pete Posted September 21, 2018 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
dotnetic Posted October 9, 2018 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.
tpr Posted October 9, 2018 Author 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.
tpr Posted October 10, 2018 Author 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
MilenKo Posted October 26, 2018 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?
tpr Posted October 28, 2018 Author 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.
adrian Posted November 22, 2018 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.
tpr Posted November 23, 2018 Author 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
adrian Posted November 23, 2018 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
adrian Posted December 19, 2018 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.
dotnetic Posted December 19, 2018 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.
dotnetic Posted December 19, 2018 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.
tpr Posted December 19, 2018 Author 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
adrian Posted December 19, 2018 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.
dotnetic Posted December 20, 2018 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?
tpr Posted December 20, 2018 Author 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.
bernhard Posted December 23, 2018 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?
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