adrian

Tracy Debugger

Recommended Posts

@gmclelland - I think you're issues should be fixed in the latest version (thanks for PM help with this).

@bernhard - I have modified the behavior of the large/small buttons in the Console panel to be fullscreen/halfscreen. Please take a look and let me know if you think that behavior suits your needs. I am ok with adding it to other panels as well if you think it works well, although obviously some panels would look pretty silly fullscreen :)

As for the z-index issues - I think that's maybe an AOS conflict - @tpr was working on that yesterday. Please let me know if they persist with AOS disabled.

Everyone else - while fixing the issue that @gmclelland was having, I make quite a few changes to the the Field List & Values section of the Request Info panel. It should now generally be even lighter and yet also show more depth of details. It also now shows unformatted and formatted values for each field. There is also a new Image Details column - displaying the thumbnails is optional (check config settings - you may want to disable if you have image fields with lots of images).

image.thumb.png.109622e46c5c87e126e0296843176aa8.png

Please let me know if you have any issues with the new version or ideas for improvements.

 

  • Like 3
  • Thanks 1

Share this post


Link to post
Share on other sites

Great job @adrian!  I'm just confirming that the update did get rid of the error I was seeing.

I like the new enhancements and how the image is displayed.  What do you think about the thumbnail linking to the original file?

  • Like 1

Share this post


Link to post
Share on other sites

That Field Values panel is huge :) I guess it's available for a while but I haven't seen it (probably along with many others) :)

Here is a small tweak that could make such large tables easier to read - sticky table headers:

.tracy-panel th {
  position: sticky;
  top: -13px;
}

sticky-th.thumb.gif.e1db7d6eebd9b5f451270bde23ac25f4.gif

The value -13px is probably a margin issue, and the inverted TH colors are there for better separation.

Of course these sticky headers could be enabled separately for the different panels but I think they won't do any harm when applied globally but maybe I'm wrong.

And Now for Something Completely Different

Rollup panel on double-click on their titles:

doubleclick-toggle.gif.7ae3a62a43c64b1e8b72008efc1eb9bc.gif

This is only quick devtools implementation but if you're interested I can roll down a better one :)

  • Like 5

Share this post


Link to post
Share on other sites
19 minutes ago, gmclelland said:

What do you think about the thumbnail linking to the original file?

Sure, I'll add to the next version.

6 minutes ago, tpr said:

Here is a small tweak that could make such large tables easier to read - sticky table headers

Sounds like a good idea - thanks!

6 minutes ago, tpr said:

Rollup panel on double-click on their titles

I think this looks awesome actually. I was kinda looking for something along these lines, but never really figured out the best approach. I went with the ESC to close all panels, but then there was no way to restore. I was going to add a Tab key option to work like tool panels in Adobe products, but you can't really make use of the tab key like that because it's needed for so many other things. I think what you have here looks like a great option for quickly getting a panel out of the way without closing it. Would appreciate some code whenever you get around to it - thanks!

  • Like 1

Share this post


Link to post
Share on other sites

Of course several unforseen issues came up but most of them are fixed:

  • closing a rolled-up panel and re-activating it from the debug bar opened it in the rolled-up state. To fix, on closing a rolled-up panel "tracy-mode-rollup" class is removed so it opens up in normal state.
  • setting width-height were tricky here and there but seems to be OK now with all the panels I've tried (Chrome only though).
  • "ProcessWire Logs" panel has a link in h1, I've added pointer-events: none in rolled-up state to prevent accidental click on move
  • resized panels were loaded cropped after page reload (if they were rolled up before reload). I've fixed it with a "beforeunload" event to allow Tracy save positions correctly
  • keeping state after page reload: would be nice to have but I think this would be better handled in the core (Nette Tracy)

Changes:

style.css

Lines at the bottom + added "user-select: none" on line 111 for h1

TracyDebugger.module

Lines 823-867 - I haven't found a better place to add the JS part, I thought there's a js file that the module always loads but apparently there isn't.

tracy-rollup.zip

  • Like 3

Share this post


Link to post
Share on other sites

Hey @tpr - thanks for your work on this - I think it's a great addition.

Not sure why, but I had to change max-height for the rolled up version from 39px to 47px, otherwise I got a scrollbar in the header. Does it look OK for you with this change (just committed).

I moved the js into main.js (which is new) as a place to load all general JS (that doesn't need injected PHP vars).

The last version also includes some z-index fixes (introduced recently in the Tracy core), your sticky table header, and I added the fullscreen/halfscreen button to a couple of other panels that I thought could benefit from it. Also added @gmclelland's idea of linking to fullsize version of image. In the PW admin this opens in a modal, but just same page in frontend.

Thanks again!

  • Like 4

Share this post


Link to post
Share on other sites

Thanks, works fine here. The 47px vs 39px issue may come from a box-sizing border-box/content-box issue, perhaps my frontend CSS was interfering somehow. If so, .tracy-panels should get box-sizing: border-box in Tracy.

Sticky table headers are working fine too, thanks.

What do you think about making the panel headers a bit smaller? I feel they are a bit too large, a  slightly slower font and less padding wouldn't hurt imho.

  • Like 1

Share this post


Link to post
Share on other sites
4 minutes ago, tpr said:

What do you think about making the panel headers a bit smaller? I feel they are a bit too large, a  slightly slower font and less padding wouldn't hurt imho.

+1 :) I am all for slim GUI for development tools...

  • Like 1

Share this post


Link to post
Share on other sites
19 hours ago, adrian said:

I have modified the behavior of the large/small buttons in the Console panel to be fullscreen/halfscreen. Please take a look and let me know if you think that behavior suits your needs. I am ok with adding it to other panels as well if you think it works well, although obviously some panels would look pretty silly fullscreen :)

awesome! this helps a lot, thanks :)

  • Like 2

Share this post


Link to post
Share on other sites
7 hours ago, tpr said:

What do you think about making the panel headers a bit smaller? I feel they are a bit too large, a  slightly slower font and less padding wouldn't hurt imho.

Sounds good, now what about the border width: 4, 3, 2, or 1?

I am thinking 2 or 3

image.png.7967a1e4819ae6a74498c886282e546c.png

image.png.63604391f7f82ea949b59c823c85d721.png

image.png.14eebb87bf57562df47384d6fa5f156c.png

image.png.5c21c83e0e942f950d2bf9405d1725d3.png

 

 

  • Like 1

Share this post


Link to post
Share on other sites

Yepp, the border is a bit thick now. I'm fine with v4 if you think panels doesn't merge too much with the rest of the site.

One thing though: the icon positions need to be adjusted (before the title and the window/close icons too) + the newly added rollup state needs adjustments too.

  • Like 1

Share this post


Link to post
Share on other sites

Hey @adrian,

what do you think of adding a check if debugInfo is available for the requested object and if not, set it to "false" automatically?

I'm working on my RockFinder module and wanted to debug my subclass. This was the result:

5ad47946b6454_2018-04-1612_20_18-ModulesProcessWirerockfinder_test.png.e0267c434c3d6fd07fd3e84078647597.png

Switching "debugInfo" to false I get this:

5ad4794a17156_2018-04-1612_19_43-ModulesProcessWirerockfinder_test.png.f8c096810386a145bf0df467a3f2bab1.png

Which is of course much more helpful. Luckily I remembered about that because we were talking about it here, but if someone missed that it could be quite hard to find out what is going on and why the information does not show up...

Also it is quite annoying to always set ['debugInfo' => false] for every quick d($myobject)

  • Like 1

Share this post


Link to post
Share on other sites

Hey @bernhard - I think it should already work that way:

image.png.9513aad49d69547b4adb94cd8f792e40.png

So I am wondering if it's due to your class extending a PW class that does implement __debugInfo()

Could you please let me know what you trying to dump in your module so I can reproduce what you are seeing. 

  • Like 1

Share this post


Link to post
Share on other sites

Ok, thx. My class is indeed extending Wire... Maybe there is a possibility to check for the number of properties the debugInfo object returns and in case it is 0 it falls back to the original object dump?

Share this post


Link to post
Share on other sites
2 minutes ago, bernhard said:

Ok, thx. My class is indeed extending Wire... Maybe there is a possibility to check for the number of properties the debugInfo object returns and in case it is 0 it falls back to the original object dump?

Maybe - I'll see what can be done, but to make it a little easier, can you let me know what variable in RockSqlFinder you are dumping, and where?

  • Like 1

Share this post


Link to post
Share on other sites

I'm doing a complete rewrite (again :P ). What I have so far is this:

<?php namespace ProcessWire;
class RockFinderField extends Wire {

  private $name; // name of the final object property
  private $field; // name of the field to query
  private $closure = null; // closure to calc value

  /**
   * constructor
   */
  public function __construct($name, $field) {
    // set the final column name for this field
    if(is_string($name)) $this->name = $name;
    else $this->name = $field;

    // check the field type
    if(is_callable($field) AND !is_string($field)) {
      $this->closure = $field;
    }
    else {
      $this->field = $field;
    }
  }
}

And in the module I do this:

foreach($fields as $key => $value) $this->fields[] = new RockFinderField($key, $value);
bd($this->fields);

 

Share this post


Link to post
Share on other sites

@bernhard - I have a temporary solution for you. Replace https://github.com/adrianbj/TracyDebugger/blob/93fe130026244fe727e5553b82183a85b78e2c9f/tracy-master/src/Tracy/Dumper.php#L460

with:

return count((array) $obj->__debugInfo()) ? $obj->__debugInfo() : (array) $obj;

There are other ways this could be achieved, but since this is modifying the Tracy core, I am going to ask them what they think is the best approach.

Let me know if it works as expected for you.

  • Like 1
  • Thanks 1

Share this post


Link to post
Share on other sites

thx, this works :) looking forward to hearing their feedback!

  • Like 1

Share this post


Link to post
Share on other sites

@adrian Morning,

I've been having a hard time with the panels since you introduced the "resize feature". I use AOS with sticky header enabled in Reno. The panels keep moving under Reno's sticky header, making it impossible to move them anywhere, as the head of the panel is "covered" by the admin's header.

There are various occasions when this happens but here is a frequent example: I use bd() and click on the arrow to expand an object with a long list. The panel expands too, moving its head under the sticky header of Reno. BTW: I'm currently running AOS 1.9.0 and Tracy 4.10.18, so these are the latest.

Before you introduced the "resize feature" I had never had such issues. Is there anything that can be done? Maybe @tpr can do something? :)

See image:

Spoiler

tray-panel-under-sticky-header.thumb.png.502cc28a93633850550d6182a2387ea2.png

 

Share this post


Link to post
Share on other sites

Hey @szabesz - I can reproduce, but it's fine with the default and Uikit themes with sticky headers so I think it's something that @tpr might need to fix. I know he made some recent changes on this front so maybe those changes need to be applied to the reno sticky header as well?

  • Like 1

Share this post


Link to post
Share on other sites

Thanks Adrian, let's wait for Roland. In the meantime I figured that making the browser window narrow removes the sticky header ("mobile view"), and I can at least drag the panel this way.

Share this post


Link to post
Share on other sites

I haven't found a way to make all the z-indexes work nicely together. I could add #tracy-debug { z-index: 100000; } or so when sticky headers are enabled in AOS but that would make menu dropdowns and such overlapped by Tracy panels.

Share this post


Link to post
Share on other sites
1 minute ago, tpr said:

I haven't found a way to make all the z-indexes work nicely together. I could add #tracy-debug { z-index: 100000; } or so when sticky headers are enabled in AOS but that would make menu dropdowns and such overlapped by Tracy panels.

But sticky headers are working fine with Tracy panels in the default and UiKit themes. Is there something different about the reno sticky header that you can't change to be like the other two?

Share this post


Link to post
Share on other sites

There are lot more elements in Reno, including the sidebar and many header elements. Setting #wrap { z-index: 0 } also solves the panel issue but still the panels are above menus.

Share this post


Link to post
Share on other sites

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 account

Sign in

Already have an account? Sign in here.

Sign In Now

  • Recently Browsing   0 members

    No registered users viewing this page.

  • Similar Content

    • By BitPoet
      MediaLibrary
      Update: MediaLibrary can now be found in the official module list.
      Out of necessity, I've started to implement a simple media library module.
      The basic mechanism is that it adds a MediaLibrary template with file and image fields. Pages of this type can be added anywhere in the page tree.
      The link and image pickers in CKEditor are extended to allow quick selection of library pages from dropdowns. In the link picker this happens in the MediaLibrary tab, where you can also see a preview of the selected image. In the image picker, simply select a library from the dropdown at the top, everything else is handled by standard functionality.
      I've put the code onto github. This module is compatible with ProcessWire 3.
      Steps to usage:
      Download the module's zip from github (switch to the pw3 branche beforehand if you want to test on PW 3.x) and unpack it into site/modules Click "Modules" -> "Refresh" in the admin Click "Install" for MediaLibrary For testing, create a page with the MediaLibrary template under home (give it an expressive title like 'Global Media') and add some images and files Edit a differnt page with a CKEditor field and add a link and an image to see the MediaLibrary features in action (see the screencap for details) Optionally, go into the module settings for MediaLibrary Note: this module is far from being as elaborate as Kongondo's Media Manager (and doesn't plan to be). If you need a feature-rich solution for integrated media management, give it a look.
      Feel free to change the settings for MediaFiles and MediaImages fields, just keep the type as multiple.
      There are some not-so-pretty hacks for creating and inserting the correct markup, which could probably be changed to use standard input fields, though I'm a bit at a loss right now how to get it to work. I've also still got to take a look at error handling before I can call it fit for production. All feedback and pointers are appreciated (that's also why I post this in the development section).

      Edit 09.03.2016 / version 0.0.4: there's now also a "Media" admin page with a shortcut to quickly add a new library.

      Edit 01.05.2016:
      Version 0.0.8:
      - The module now supports nested media libraries (all descendants of eligible media libraries are also selectable in link/image picker).
      - There's a MediaLibrary::getPageMediaLibraries method you can hook after to modify the array of available libraries.
      - You can switch between (default) select dropdowns or radio boxes in the module configuration of MediaLIbrary to choose libraries.
      Edit 10.10.2018:
      Version 0.1.3:
      - Dropped compatibility for ProcessWire legacy versions by adding namespaces
      - Allow deletion of libraries from the Media overview admin page
      - Added an option to hide media libraries from the page tree (optionally also for superusers)
    • By Robin S
      This module corrects a few things that I find awkward about the "Add New Template" workflow in the PW admin. I opened a wishlist topic a while back because it would good to resolve some of these things in the core, but this module is a stopgap for now.
      Originally I was going to share these as a few standalone hooks, but decided to bundle them together in a configurable module instead.
      Add Template Enhancements
      A module for ProcessWire CMS/CMF. Adds some efficiency enhancements when adding or cloning templates via admin.

      Features
      Derive label from name when new template added: if you like to give each of your templates a label then this feature can save some time. The label can be added automatically when templates are added in admin, in admin/API, or not at all. There are options for underscore/hyphen replacement and capitalisation of the label. Edit template after add: when adding only a single template, the template is automatically opened for editing after it is added. Copy field contexts when cloning: this copies the field contexts (a.k.a. overrides such as column width, label and description) from the source template to the new template when using the "Duplicate/clone this template?" feature on the Advanced tab. Copy field contexts when duplicating fields: this copies the field contexts if you select the "Duplicate fields used by another template" option when adding a new template. Usage
      Install the Add Template Enhancements module.
      Configure the module settings according to what suits you.
       
      https://github.com/Toutouwai/AddTemplateEnhancements
      https://modules.processwire.com/modules/add-template-enhancements/
    • By Mike Rockett
      As I mentioned in this issue, I've create a new textformatter for ParsedownExtraPlugin, which adds some oomph to your markdown.
      Repo: Parsedown Extra Plugin
      Unlike the built-in textformatter for Parsedown and Parsedown Extra, this should be used when you want to use Extra with additional configuration/customisation.
      Some examples:
      ### Test {.heading} - A [external link](https://google.com/){.google} with `google` as a class that opens in a new tab if the config property is set. - [Another link](/page){target=_blank} that opens in a new tab even though it isn't external. ```html .html <p>Test</p> ``` There's some config options available to you, such as setting attributes on all/external images and links, setting table and table-cell alignment classes, adjusting footnote classes and IDs, adding <code> attributes to their parent <pre> elements, and changing the <code> class if your syntax highlighter does not use language-*.
      I was thinking about adding the ability to make links open in a new tab by appending a plus to the link syntax, but only external links should be opening in a new tab anyway. Further, this would add extra, unnecessary processing time.
      Please let me know if you bump into any problems. ☺️
    • By Mike Rockett
      TextformatterTypographer (0.4.0 Beta)
      A ProcessWire wrapper for the awesome PHP Typography class, originally authored by KINGdesk LLC and enhanced by Peter Putzer in wp-Typography. Like Smartypants, it supercharges text fields with enhanced typography and typesetting, such as smart quotations, hyphenation in 59 languages, ellipses, copyright-, trade-, and service-marks, math symbols, and more.
      Learn more on my blog
      It's based on the PHP-Typography library found over at wp-Typography, which is more frequently updated and feature rich that its original by KINGdesk LLC.
      The module itself is fully configurable. I haven't done extensive testing, but there is nothing complex about this, and so I only envisage a typographical bug here and there, if any.
      Please do test it out and let me know what you think.
      Also note that I have indicated support for PW 2.8, but I haven't tested there as yet. This was built on PW 3.0.42/62.
    • By Mike Rockett
      Jumplinks for ProcessWire
      Release: 1.5.50
      Jumplinks is an enhanced version of the original ProcessRedirects by Antti Peisa.
      The Process module manages your permanent and temporary redirects (we'll call these "jumplinks" from now on, unless in reference to redirects from another module), useful for when you're migrating over to ProcessWire from another system/platform. Each jumplink supports wildcards, shortening the time needed to create them.
      Unlike similar modules for other platforms, wildcards in Jumplinks are much easier to work with, as Regular Expressions are not fully exposed. Instead, parameters wrapped in curly braces are used - these are described in the documentation.
      Under Development: 2.0, to be powered by FastRoute
      As of version 1.5.0, Jumplinks requires at least ProcessWire 2.6.1 to run.
      View on GitLab
      Download via the Modules Directory
      Read the docs
      Features
      The most prominent features include:
      Basic jumplinks (from one fixed route to another) Parameter-based wildcards with "Smart" equivalents Mapping Collections (for converting ID-based routes to their named-equivalents without the need to create multiple jumplinks) Destination Selectors (for finding and redirecting to pages containing legacy location information) Timed Activation (activate and/or deactivate jumplinks at specific times) 404-Monitor (for creating jumplinks based on 404 hits) Additionally, the following features may come in handy:
      Stale jumplink management Legacy domain support for slow migrations An importer (from CSV or ProcessRedirects) Feedback & Feature Requests
      I’d love to know what you think of this module. Please provide some feedback on the module as a whole, or even regarding smaller things that make it whole. Also, please feel free to submit feature requests and their use-cases.
      Note: Features requested so far have been added to the to-do list, and will be added to 2.0, and not the current dev/master branches.
      Open Source

      Jumplinks is an open-source project, and is free to use. In fact, Jumplinks will always be open-source, and will always remain free to use. Forever. If you would like to support the development of Jumplinks, please consider making a small donation via PayPal.
      Enjoy!