matjazp

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

Yep. But it's hard to work on things that are working and you are trying to make them not working just do debug them ?

I was suspecting locale problems from the very beginning, that's why I asked to dump basename() and pathinfo() calls since they are both locale aware. Just to test again, could you @PWaddict try this: d(basename("ä")); with and without translated "C" in LanguageSupportModule? Though I'm getting the correct results regardless of "C" value. That's why we have a check that is performed on every login (and some changes are introduced recently to better clarify the issue here)! Do you get a warning message when you log in, something like "Note: your current server locale setting isn’t working as expected with the UTF-8 charset and may cause minor issues. Your current locale setting is “en_US.UTF-8”. Please translate the “C” locale setting for each language to the compatible locale in ... " ? if not, then why you translated the "C" in the first place and also translated to the nonexisting locale? There is no need to translate "C" string on windows (from my experience). Windows don't have "en_US.UTF-8" locale, that's why PW suggest "en-US". The list of supported locales on windows: https://msdn.microsoft.com/library/cc233982.aspx , see the language tag column. As for getting mixed results: the locale information is maintained per process, not per thread. If you are running PHP on a multithreaded server API like IIS, HHVM or Apache on Windows, you may experience sudden changes in locale settings while a script is running, though the script itself never called setlocale(). This happens due to other scripts running in different threads of the same process at the same time, changing the process-wide locale using setlocale(). Glad you solved the problem ?

Hm, I'm running out of ideas... And this? d($sanitizer->filename($filename)); d($sanitizer->filename($filename, Sanitizer::translate));

Now that's odd. Please do: d(pathinfo($filename)); on both affected instalations. @adrian off topic: when I do bd() call in Sanitizer.php in the name() method (line 245) I get: Fatal error: Uncaught Error: Call to undefined function ProcessWire\bd() in C:\inetpub\wwwroot\wire\core\Sanitizer.php:246

I get the same results as Adrian. Are there any hooks in the game? Tried on fresh install? Could you also d(basename($filename)); @PWaddict

I'm on windows:

I used PW upgrades module... Edit: Adminer 4.7.0 (released 2018-11-24)

Just wanted to upgrade her, but I got compile error: "Cannot declare class TracyDebugger, because the name is already in use", File: ...\modules\TracyDebugger\TracyDebugger.module.php:24

@bernhard would you mind editing WireFileTools.php and in rmdir() method insert $path = rtrim($path, "/"); just before $files = scandir($path);

I know, I opened it ? Ryan tried to fix this issue, but introduced another one. Try: $files->mkdir($config->paths->cache . "test/test/", true); $files->rmdir($config->paths->cache . "test/", true, array('throw' => true)); By default throw is false, I set it to true just for you/others to see the results. The point is: rmdir() fails to remove a directory that contains subdirectories due to a bug. Edit: ?