tpr Posted March 23, 2017 Author Posted March 23, 2017 Yes, that's an issue that needs some thinking. Do you think globally disabling to remember position would be enough? Also now all splitters share the same splitter setting which is probably not suitable for everyone. I guess I can fix this easily though. Another idea borrowed from file managers to use double click on the splitter to restore the default position. So you can drag it freely and it would be remembered but you could restore defaults when needed. 1
adrian Posted March 23, 2017 Posted March 23, 2017 21 minutes ago, tpr said: Do you think globally disabling to remember position would be enough? I think the key thing is that when there is no position stored in LocalStorage, it should use the field's width setting (default or template context override) as it did before. 22 minutes ago, tpr said: Another idea borrowed from file managers to use double click on the splitter to restore the default position. So you can drag it freely and it would be remembered but you could restore defaults when needed. Yeah, something like that sounds good.
tpr Posted March 23, 2017 Author Posted March 23, 2017 Ok, all shoud be fine in v139. One thing that I removed is the drag handler icon. I couldn't find a way to show it nicely so now the only thing that indicates the splitter is the mouse cursor. There was an issue with the splitter's height too. Now it's set by JS because I didn't want to go flex and risk breaking other things. From the docs: Quote From v137 columns are resizable by dragging the gutter between the columns. The splitter position is remembered per template and per user. Double-click on the splitter to restore the default position (set at aos_column_break field's "Column Width" setting, or defaults to the module's default value). This feature is not available inside tabs. 1
adrian Posted March 23, 2017 Posted March 23, 2017 2 minutes ago, tpr said: Ok, all shoud be fine in v139. One thing that I removed is the drag handler icon. I couldn't find a way to show it nicely so now the only thing that indicates the splitter is the mouse cursor. There was an issue with the splitter's height too. Now it's set by JS because I didn't want to go flex and risk breaking other things. Looks and works great now - thanks! 1
Robin S Posted March 24, 2017 Posted March 24, 2017 Hi @tpr, I just noticed that the position of the restore icon for PageListSelect looks a bit off when the field is inside a repeater:
tpr Posted March 24, 2017 Author Posted March 24, 2017 Thanks, please try replacing the main CSS. AdminOnSteroids.css 1
bernhard Posted March 24, 2017 Posted March 24, 2017 hi tpr, the draggable devider looks great. would it be possible to add this feature also to the core pw-panel?
bernhard Posted March 24, 2017 Posted March 24, 2017 I'm talking about the built in panel. Just add the class "pw-panel pw-panel-right" to a link and it opens in a sidebar panel like this: It's similar to the modal, but newer and i prefer it over modals. It's great to open PDFs for a preview for example. Or to open items in my datatables. See \wire\modules\Jquery\JqueryUI\panel.js for details. Would be awesome to make those panels draggable
tpr Posted March 24, 2017 Author Posted March 24, 2017 The "Tree" panel is already draggable here, and also the "View page" panel. But it's a different thing than in case of columns, here you only resize one element. 1
Robin S Posted March 25, 2017 Posted March 25, 2017 19 hours ago, tpr said: Thanks, please try replacing the main CSS. AdminOnSteroids.css That fixes it, thanks. 1
bernhard Posted March 26, 2017 Posted March 26, 2017 On 24.3.2017 at 3:32 PM, tpr said: The "Tree" panel is already draggable here, and also the "View page" panel. But it's a different thing than in case of columns, here you only resize one element. thanks for making me aware of this! works only on left-aligned panels though... https://github.com/processwire/processwire/blob/50f2834a0ae2a9f9f2ac53439184ebdc04db2f54/wire/modules/Jquery/JqueryUI/panel.js#L411-L432 better than nothing
tpr Posted March 30, 2017 Author Posted March 30, 2017 Handy shortcuts to edit translations in a textdomain file in other languages in the upcoming version: 2
tpr Posted March 31, 2017 Author Posted March 31, 2017 The abovementioned feature is available in v141, plus a new CKEditor plugin Indent Block and some minor updates. 4
Robin S Posted April 2, 2017 Posted April 2, 2017 @tpr, I often want to work with system templates. Examples being: Add fields to the user template, and want easy access to the template from the admin menus or without having to set the "Show system templates" filter in the templates list. When editing a field, add that field to the template of a repeater field. Normally to do stuff like this you have to have $config->advanced = true in /site/config.php, but I don't want the risk of messing up the other things that this setting exposes. So I looked at making system templates visible through hooks and came up with this: // Show system templates $this->addHookBefore('ProcessField::buildEditFormInfo', function($event) { // Show system templates in templates list of Edit Field $this->wire('config')->advanced = true; }); $this->addHookBefore('ProcessTemplate::executeNavJSON', function($event) { // Show system templates in admin menu $this->wire('config')->advanced = true; }); $this->addHookAfter('Session::loginSuccess(roles=superuser)', null, function($event) { // Show system templates in templates list unless explicitly hidden $this->wire('session')->set('ProcessTemplateFilterSystem', 1); }); Is this something you think would be a good option to add to AOS? 2
szabesz Posted April 2, 2017 Posted April 2, 2017 7 minutes ago, Robin S said: would be a good option to add to AOS +1
tpr Posted April 2, 2017 Author Posted April 2, 2017 8 hours ago, Robin S said: Is this something you think would be a good option to add to AOS? Sure, I'll have a look later. 8 hours ago, Robin S said: $this->addHookAfter('Session::loginSuccess(roles=superuser)', null, function($event) { This "(roles=superuser)" is new to me. Is there somewhere I can read more about this (and similar tricks)?
bernhard Posted April 2, 2017 Posted April 2, 2017 https://processwire.com/blog/posts/new-ajax-driven-inputs-conditional-hooks-template-family-settings-and-more/#new-conditional-hooks 4
Robin S Posted April 2, 2017 Posted April 2, 2017 @tpr, another idea/request: if the "permanent delete" option for Page List is checked then there is also an option to permanently delete the page from the Delete tab of Page Edit. Thanks! 2
szabesz Posted April 3, 2017 Posted April 3, 2017 How about: Confirm to move to Trash Permanently delete instead to make it less confusing?
adrian Posted April 3, 2017 Posted April 3, 2017 Hey @tpr - as I mentioned here: https://github.com/ryancramerdesign/AdminThemeUikit/issues/20, the icons only PageList option doesn't work in the UiKit theme sidebar. Now maybe Ryan will add this option to the core, but if not, it would be good to get fixed in AoS. Also, the full width hover links don't work. And also wondering if you can make an icon for the template link in the PageList - this would also help with allowing the width of the sidebar to be narrower. I understand that the UiKit theme is still a moving target, so just ideas for you at the moment
tpr Posted April 5, 2017 Author Posted April 5, 2017 I'll have a look on the above issues/requests later, nowadays I'm busy with a 4.04 kg newborn who thinks he can steal all the hours in a day (and night) 7 1
bernhard Posted April 5, 2017 Posted April 5, 2017 1 minute ago, tpr said: 4.04 kg newborn That sounds like a baby on steroids!!! Congratulations! 3
tpr Posted April 5, 2017 Author Posted April 5, 2017 Sure It's the second and I thought the first one is on steroids... Now I know I was very wrong 5
szabesz Posted April 5, 2017 Posted April 5, 2017 6 hours ago, tpr said: It's the second and Keep up the good work! You need to keep your sanity so that you will also be able to have fun with the third one too 1
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now