Jump to content

adrian

PW-Moderators
  • Posts

    11,213
  • Joined

  • Last visited

  • Days Won

    373

Everything posted by adrian

  1. I like both actually - the first option really does look a lot nicer than the grey which is pretty awful now that I see the difference ? The second one is nice but you're right, it will be problematic for the orange and red icons. Anyone have any thoughts? Not sure yet - I don't think I can hook into debugInfo though - I'll see what I can do about this sometime soon.
  2. I am getting a little annoyed with the debugInfo output myself actually - there are lots of holes in it's output - there is a bit of a discussion about it here: https://github.com/processwire/processwire-issues/issues/575 but it would be a lot of work for Ryan to go through all of PW and set this up properly. Of course there is a config setting to make the default behavior show the full PW object, but there are times when the debugInfo output is better/easier. I mentioned above somewhere that I was thinking about having an option to dump both versions automatically with any dump/bardump call - best of both worlds. What do you think about this as an alternative? Not saying I don't like the verbose idea - I just want to make sure we don't end up with more options than we need. It might also be confusing if you already have debugInfo off by default in the config settings. I don't know - anyone else have any thoughts?
  3. Thanks! Also, just an FYI that your bdb and db 999 has been changed to 9999 in that version.
  4. New version just committed contains all the icon color changes. Mostly it's just consolidating the default color to grey and changing a few instances of when orange (WARN) vs red (ALERT) is used. Let me know if you find anything I have missed or any inconsistencies.
  5. @Guy Verville - I just updated the module with a more reliable fix - the other one would probably have worked on almost all sites, but there was a possibility for an issue. Please let me know if you have any problems with the new version.
  6. Works great now - thanks!
  7. Ok, I have been looking at the admin live search results and it seems that if the user doesn't have edit rights (either in general, or because of ARB), then the result link goes to view the page on the frontend, rather than edit in the backend. I think I find it a bit strange that the results don't have links to both when appropriate. But anyway, I don't think there are actually any issues with ARB and the new PW admin search, but I wouldn't mind a few others to investigate to make sure I haven't missed anything.
  8. Hey @Robin S - could you please take a look at this? I am seeing it if I try to edit a page that I don't have permission to edit. I am using AdminRestrictBranch, but I expect it might happen regardless of this. Let me know if you need any help reproducing. Thanks!
  9. I have pushed a fix for this, but I have discovered that Ryan's recent changes to the admin live search functionality has broken restricting of search results. I am working on this now, but could be a bit painful.
  10. Thanks - I see the problem - it's to do with the way ARB blocks pages when viewed in Lister and because the Users page in PW uses Lister, it is causing the problem. I should have a fix shortly.
  11. Glad I stumbled across this post - ARB should not get involved with displaying users - is there anything special I need to do to reproduce?
  12. Just the default icons to give you guys an idea, but how's this looking? Still not sure if Debug Mode icon on should be orange or red? Perhaps I should check if on local dev or live server and go with grey for the former and red for the latter where it really is important!
  13. Honestly I am starting to feel like this is something that should be solved by the PW core - I don't really feel like messing around with hacky solutions for this. @iank - would you be willing to submit an issue to the PW github account? I am honestly not sure if this should be an issue or a request, but I feel like it's strange that normal fields allow for an easy fallback, but module settings don't, so maybe it's an issue?
  14. Sounds good - I don't think you need to go creating them - probably just need to spend some time looking around for better free ones, but whatever you think would be great. I think regardless of how you go about creating or finding them we should wait till you have them all ready before committing any changes.
  15. Also, if there are any icon geeks out there, I'd love some help to revamp the icons on the debug bar - many of them are very ugly ? Any takers?
  16. I just wanted to clarify that the grey/green/orange/red color scheme is already in place and has been since the beginning of the module. As per my mockup above I am more than happy to make all the icons that don't need this scheme grey instead of PW pink or PHP purple, or blue (the System Info icon). Hopefully we're all good with that change. I think this approach of grey instead of green to indicate no attention needed will be fine - I can't currently see any reason not to make this change. I think there will always be some difficulties in determining what should be orange and what should be red, although I do think that if we get rid of green altogether it will actually be simpler, for example I always felt a bit weird about debug mode set to on resulting in red. I think a new grey / orange only combo is probably much nicer. Orange being a warning that you need to be aware of the fact that it's on and this is a security concern on a live site. No problem at all - as with most things, Tracy has evolved and wasn't really planned from the beginning, so more than happy to revisit this stuff now and I really appreciate everyone's input.
  17. Hi @iank - sorry, I have no idea how ProCache will handle this - I have never used it. Regarding dealing with fallback if empty - I think the only way to logically do this and also consider @tpr's point about allowing for an intentional blank value is maybe to only fallback when a field is required. I think that will work fine for this module. What do you both think?
  18. Doesn't say much, but #3 on a list of the 6 best alternatives to Drumlapress: https://www.geckoandfly.com/27533/wordpress-drupal-joomla-alternatives/
  19. Hi everyone - @netcarver has mentioned to me that he's not a fan of some of the ambiguity of the icon colors in the debug bar - mostly the blue for the System Info icon and the PW pink color used by the PW Info and Captain Hook panels, and the purple used by the PHP Info panel. I have quickly mocked up a version where these are all grey: He wanted me to poll you guys to get some input on this change and also any other changes you might like to see. In general the idea with icon colors is: grey - no significance green - everything ok - no attention needed orange - warning or attention to a new entry/item of interest red - warning, fail, or something else that you should be aware of So please post your thoughts - what do you think can be done to make it look cleaner and also more meaningful/useful? Thanks!
  20. Typically the simple solution for this would be to drag the Tracy debug bar out of the way - this will work so long as you don't have the "Fixed Position" option checked in the config settings.
  21. I had the same thought actually - thanks for bringing it up here. I do feel like in particular with module settings that it's something that shouldn't be hard to populate all languages with - it's not the same as new pages added to a site where you really can't enforce a site editor to populate all the alternate language values.
  22. Thanks for testing. I have done the same here and can confirm your results. The problem is that I don't think PW has an inbuilt method for handling fallbacks for module settings. Because modules settings don't return an object you need to explicitly get the required value based on the user's language. I could write something custom in this module but I am curious if any of the ML experts out there have dealt with this when they have module config fields that have: 'useLanguages' => true
  23. I have looked into OpenCollective and even suggested it to @tpr for AOS - the problem is that they require you to have 100 Github stars on a project before they'll accept it. So if you guys want to help out, send some star loving ? Obviously the PW core could get involved, but I don't think there are any modules with 100 stars yet.
  24. Hi @martind - sorry about that. I am not sure when this started happening, but I feel like PW's file compiler is no longer correctly dealing with adding the namespace to classes in modules. Anyway, I have implemented a fix - please check the latest version and let me know how it goes.
×
×
  • Create New...