mindplay.dk Posted November 7, 2012 Share Posted November 7, 2012 This is just a tiny, one-line tweak - but it really makes a huge difference in terms of ease and speed in working with the Page List. I've added this to my admin theme, but I really think this belongs in the core. in "ProcessPageList/ProcessPageList.css": /** * When an item is open, this style ensures that the PageListActions become visible * */ .PageList .PageListItemOpen > ul.PageListActions, .PageListItem:hover > ul.PageListActions, /* <-- add this line! */ .PageListSelectHeader .PageListActions { display: inline !important; } You now have direct access to "new", "edit", etc. - just by hovering over an item in the Page List. Give it a try - you will most likely never want to go back! 6 Link to comment Share on other sites More sharing options...
mindplay.dk Posted November 8, 2012 Author Share Posted November 8, 2012 I personally also added a rule that hides the PageListActions for expanded Pages - so that the buttons display only when you hover over an item. That may be taking it too far for some people's taste, but I find that it minimizes the amount of noise on the page greatly - I don't need a screen full of buttons. That may be more of a personal matter though 1 Link to comment Share on other sites More sharing options...
netcarver Posted November 8, 2012 Share Posted November 8, 2012 I've tried this and like it. The idea in your second post sounds like a good addition too. However, I do have one thought: would it be even nicer if the hover target was narrower? Currently the list items are pretty wide so the menu pops up even if the cursor is over the other side of the screen which seems a little over-animated to me. BTW, I'm using the default theme - I guess other themes may have a more restricted width on the list items. Link to comment Share on other sites More sharing options...
diogo Posted November 8, 2012 Share Posted November 8, 2012 I think it's a good idea, but somehow it's a bit confusing... try this: /** * When an item is open, this style ensures that the PageListActions become visible * */ .PageListItem:hover > ul.PageListActions{ display: inline !important; opacity: 0.4; } .PageList .PageListItemOpen > ul.PageListActions, .PageListSelectHeader .PageListActions { display: inline !important; opacity: 1; } Link to comment Share on other sites More sharing options...
mindplay.dk Posted November 8, 2012 Author Share Posted November 8, 2012 would it be even nicer if the hover target was narrower? It would no longer be obvious that the navigation exists - I don't think you want users to have to "search" for it. It's may be a little "over-animated" this way, but there's something to be said for usability too, not just for design... (just my $.02) @diogo - having them still showing with opacity just makes things look muddy to me - I prefer to keep the UI clean. (this, on the other hand, is a design consideration, so perhaps just my personal preference) Given all these very personal pros and cons, perhaps I was wrong - perhaps it is better to just keep this in your own admin theme? Link to comment Share on other sites More sharing options...
ryan Posted November 8, 2012 Share Posted November 8, 2012 I agree I think this does seem like a nice optimization, as you can perform an action on a page separately from having to open its children, so this ultimately saves time/resources. However, I also agree with Netcarver about the issue with hovering in the whitespace, though experimented a bit and not really sure how to solve this. Link to comment Share on other sites More sharing options...
diogo Posted November 8, 2012 Share Posted November 8, 2012 The white space thing can be solved like this .PageList .PageListItemOpen > ul.PageListActions, .PageListItem:hover > ul.PageListActions, /* <-- add this line! */ .PageListSelectHeader .PageListActions { display: inline !important; } .PageListItem{ display: inline !important; } .PageListItem:after { content:"\A"; white-space:pre; } not pretty but it works. edit: .PageListItem can go up of course, kept it separate for illustration purposes 2 Link to comment Share on other sites More sharing options...
Marty Walker Posted November 8, 2012 Share Posted November 8, 2012 If saves an extra click it's an improvement. Link to comment Share on other sites More sharing options...
netcarver Posted November 9, 2012 Share Posted November 9, 2012 @diogo, thank you, overall that seems better to me but may not be to everyone's taste and I do now see mindplay's point about the navigation not being so obvious. Link to comment Share on other sites More sharing options...
ryan Posted November 9, 2012 Share Posted November 9, 2012 ot pretty but it works. What do you mean that it's not pretty? Is there anything wrong with this method? Looks pretty darn elegant to me (nice job!), especially after I was mucking about trying to do this same thing in Javascript. Link to comment Share on other sites More sharing options...
diogo Posted November 9, 2012 Share Posted November 9, 2012 I don't know... I just think there should be a way of telling a block element to have the size of it's content... Link to comment Share on other sites More sharing options...
mindplay.dk Posted November 9, 2012 Author Share Posted November 9, 2012 I don't know... I just think there should be a way of telling a block element to have the size of it's content... display:inline-block; width:auto; margin:0 auto; ? Link to comment Share on other sites More sharing options...
diogo Posted November 9, 2012 Share Posted November 9, 2012 Ok, I didn't explain myself very well... I meant a block element with the size of it's content that doesn't allow any other element next to it. But giving it another thought, that's just not the way the flow works. Link to comment Share on other sites More sharing options...
Martijn Geerts Posted November 9, 2012 Share Posted November 9, 2012 mindplay.dk & diogo, tnx for sharing. Used this today & love it! Link to comment Share on other sites More sharing options...
mindplay.dk Posted January 15, 2013 Author Share Posted January 15, 2013 There's a problem with this tweak - if a page is not selected/open (.PageListItemOpen) you can't drag it. This appears to be caused by "ProcessPageList.js", where the clickMove() function has a block of code with the description, "make an invisible PageList placeholder that allows 'move' action to create a child below this". I'm not sure how to fix this, or if that's even precisely the problem... any ideas? Link to comment Share on other sites More sharing options...
ryan Posted January 16, 2013 Share Posted January 16, 2013 I noticed that too. At the time, I tinkered with it for a bit, but couldn't figure and gave up. Will play with it more next time I'm on there, but if anyone else finds the solution sooner let me know… Link to comment Share on other sites More sharing options...
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