Martijn Geerts
Martijn Geerts
I didn't recognise that for theme Reno there's an other class and an other method for this. So next to the AdminThemeDefaultHelpers::renderTopNavItems it would be great to have AdminThemeRenoHelpers::renderSideNavItems also...
I thought of that and have tried it but didn't work out because of the [check_access=0](https://github.com/ryancramerdesign/ProcessWire/blob/dev/wire/modules/AdminTheme/AdminThemeDefault/AdminThemeDefaultHelpers.php#L381) in the selector I guess. (or some other things I don't be aware of)...
It was stupidity on my side about the permission, your solution works. On the other hand I still have the need to change the markup anyway so the question is...
+1 We in Europe can't use comma separated as comma's are used in floats. Is it an idea to substitute the radio's with Inputfile text? (defaulting to comma?)
@ryancramerdesign, so far I see this 'bug' is still there. We now rely on a dirty patch that disables the access checking which in my opinion is a bad thing....
Interesting, there are more benefits then the valid point Soma is taken. Readability of code, inline elements don't take extra white-space space, easier to maintain.