matjazp

Hi, Thanks for this module. Nice and organized code! is there an option to disable session lifetime, like set it to 0? The minimal value is 1 due to how you check for 0. I see you are sending ajax requests to location.pathname.substring(1) + "/phcv1" but at least on my localhost location.pathname is /somepage/ so ajax is sent to http://localhost/mypage/mypage//phcv1 Shouldn't you use just location.pathname + "phcv1" ? Counter on the home page isn't working as you are passing an int to ctype_digit() function which expects a string. What I find odd is that ctype_digit(1) returns false while ctype_digit(1054) returns true. I'm not sure if ctype_digit is actually needed. The downside of this script is that sending ajax request put some load to the server itself and at least one findOne(). I'm not sure I like counters on the page list, would prefer table layout somewhere else.

But are those locales available on your server?

@bernhard thx for this. I'm playing with it, looks fine. But when I select to use default uikit theme in /wire/modules, not in /site/modules, I get this in Chrome: Refused to apply style from 'http://localhost/site/assets/RockSkinUikit/theme.less' because its MIME type ('text/ html') is not a supported stylesheet MIME type, and strict MIME checking is enabled. Because of that the look of the admin was, well...I had to uninstall RockSkinUikit to get admin theme back.

Buttons should have some bottom margin (on mobile view).

It looks like that's the problem of pagefinder and not DynamicRoles. Please try reversing the selectors, first has_parent and then id (has_parent|id=1631).

Please see the attached screenshot, Business to Consumer is written over the Blog. But not a big deal... There are still two prev posts in the footer (see attached screenshot from @gmclelland). We all agree on that. I think the examples on the site should have run button (like Codepen) so that users can run the examples and play with them, I think Tracy could fit here (I think Adrian already suggested something in that regard). @ryan While an example is good, for newbies this is odd. What is "output formating"??? I think we should use simple examples that just work. Mixing $pages and pages() is confusing too. Why don't we just stick to one notation, at least in examples, and have a special page where all these options are explained in details?

Don't be done, please. We need your valuable opinions.

I agree if that helps ? I also find it confusing why there are multiple options and what is the benefit of each...

Could you provide information on how to duplicate?

I have to agree here. While I'm ok with any font (either system or non-system) for text, I think we are losing the brand with system fonts for the logo. I think that we should keep the original logo with icon and text together. I also don't like iMac picture. For me, it doesn't make me think that this CMS is for developers, but rather "O, this software only works on Apple device, I'll pass that one". As I understand, this is just temporary?

Did you try this version: https://github.com/matjazpotocnik/DynamicRoles ?

Wouldn't just adding style="fill:#fff" or fill="#fff" to the <path element suffice? Or even: <style> .pw_logo_color {fill: #fff} .pw_logo_color:hover {fill: #D82C82} </style> <path class="pw_logo_color" ...

Testing on windows 8, IE11 and Chrome, segoe ui is fine here, on both browsers. From the list of other Typekit fonts I prefer benton-sans, but system fonts are good for me. On IE the logo (top left) is black (invert filter is not supported on IE).

When reading the latest blog post (https://processwire.com/newsite/blog/posts/rebuilding-the-pw-site-5/), there are two links at the bottom of the page to the previous blog posts. This might be by design since there is no next blog post, but it looks odd having two previous posts... I see that the hamburger icon in mobile view is now fixed, thanks Ryan. When you submit a site to the showcase list, the link contains two slashes (/newsite/sites//submit). The categories list in mobile view overlaps.

I'm browsing on my phone with javascript turned off and there are no menus and no hamburger icon.

When I try to edit a template that gets its fields from another template, I get an error: Argument 2 passed to ProcessWire\InputfieldWrapper::insertAfter() must be an instance of ProcessWire\Inputfield, null given, called in C:\inetpub\wwwroot\site\modules\TemplateFieldWidths\TemplateFieldWidths.module on line 152 Possible fix: if($fg_fields) $form->insertAfter($f, $fg_fields);

Typo inTracyDebugger.module.php, line 2688: reponse -> response.

Cool! It never crossed my mind I could clik/hoover on that icon. Form usability point of view it would make more sense to have the underlined text: "Keybord Shorcuts". Of course, that's just my opinion, not a request for you to implement it... I edited my post providing possible solution ...

If you don't RTFM you don't know there are shortcuts available. I understand why you don't want more checkboxes. I wanted to suggest text info about shortcuts, but then I discovered that you already have alt text on Run button. The problem is that I never hoovered long enough to see that. Yeah, both modules have a plethora of options, so it's expected to take some time to load. There's a lot of javascript things going on behind the scenes. I'm not a js guru, but I'm sure there is room for improvement in PW admin theme/inputfields itself. When I tried Page Files panel, I get this error: Error: Call to a member function filesManager() on integer in C:\inetpub\wwwroot\site\assets\cache\FileCompiler\site\modules\TracyDebugger\panels\PageFilesPanel.php:171 Edit: support for fieldtype FieldtypeFieldsetPage is missing, it looks lik this is working in getPageFiles and getDiskFiles... if($item && $f && $f->type instanceof FieldtypeRepeater) { $repeaterValue = $p->get($f->name); if($repeaterValue instanceof Page) $repeaterValue = array($repeaterValue); foreach($repeaterValue as $subpage) { //foreach($p->$f as $subpage) { $files += $this->getPageFiles($subpage); } }

Oh, didn't know about shortcuts! That's enough for me, I don't mind using the shortcuts, thanks.

Wish: is it possible to have a checkbox (unchecked by default) in the console panel, that would clear the results automatically before run?

Thanks for the feedback. I posted suggested solution on the github. That's because $sanitizer->(file)name() checks for multibyte support and uses mb_strtolower().

@PWaddict could you please modify $basename = strtolower($basename); to $basename = mb_strtolower($basename); in cleanBasename method in /wire/core/Pagefiles.php (line 568).

I'm still not satisfied as I can't replicate this. I would like to perform some more tests, but don't want to pollute this thread, so if you are interested in narrowing down this issue, please PM me, @PWaddict

I cant't help any more since I'm going to a vacation for a few days. It would be helpful if you find the reason for all this. Will it help if you use just "en-US" for the windows locale? Or try this script: https://www.zen-cart.com/attachment.php?s=ecb022422821941d39f3bff5511c3ecb&amp;attachmentid=16404&amp;d=1465221878