Orchard admin : todo general #7016
Comments
|
Contents -> Improve LocalizationPart rendering in SummaryAdmin mode: |
|
We could replace the Navigation UI tree jquery plugin with an Angular one. |
|
I actually prefer to stick with what comes with Amaretti as much as possible as that makes it easier to apply updates. Unless there is a compelling reason to switch to Angular UI tree? |
|
Amaretti uses jquery.nestable On second thought there is no need to replace if it works. I will just make some styling tweaks, it will be faster. |
|
We need to improve the responsive behavior of the top menu. Right now the only the "culture picker" and the "user menu" appears in it. It misses "dashboard" and "content definition" menus. I believe that the culture picker should not appear on that "hamburger" menu but should appear on the top right green portion of the header. Same goes for the user menu. Then the 2 other top left menus should appear in the "hamburger" menu. |
|
After working a little on the top menu I think we will need to be able to display the culture picker and the user menu in that "hamburger" menu. There is not enough space left on the top green portion of the header when we are on smart phone devices. Right now they are |
|
We need to do a better javascript validation on the email test form. Right now I used the new Toastr plugin for displaying validation errors but I should not have done it that way. Reason is that we can now spam that send button to get notifiers stacking one on top of the other.
I could also set the Toastr plugin to simply not allow duplicate messages. Which I will do for now. toastr.options = {
preventDuplicates: true
}; |
|
I moved the Localization to Meta properties in Summary admin. |
|
Need to refactor the implementation of panels by inheriting them instead of overriding them. See orchard-panels.less. We need to have our own panels and inherit them for reusability of the Amaretti panels. .or-panel-main{
.panel;
/* our customizations */
}This is what prevents me from using them right now in the Module / Features view. We overrided them with Flexbox and changed the default margin / padding of those. |
|
Merged the latest dev in the new admin theme branch |
|
Created a new token picker that has no dependencies on JQuery UI. I've not implemented autocomplete because most of the time the textbox that has a token picker will already have a text which makes the autocomplete list filtering useless. I used a Boostrap button dropdown with some tweaks. That way we can make the pop-up box responsive and we get rid of the bug that adds a text in the bottom of the view that indicates that we can use arrows to navigate the token picker values. Feel free to look at it on the autoroute part settings. Also by using a Bootstrap input-group with a button we no longer have to fix the margins of that icon inside the textbox for each browsers which was still clunky at some places. I used a leaf for the icon, so a token is like an Orchard leaf 😉
|
|
I needed to add a validation if there was submenu items on that function so that it not apply the "open" class to a root menu item that has no submenu. It needs to be added to the Amaretti base theme main.js file. function openSubMenu(el, event, click) {
var li = $(el).parent();
var orScroller = li.find(".or-scroller");
var subMenu = li.find("> ul");
var isOpen = li.hasClass('open');
//Show sub menu element
if (!$.isXs()) {
$("ul.visible", leftSidebar).removeClass("visible").parent().removeClass('open');
//Add classes to create a delay on sub menu close action after mouse leave
if (subMenu.length > 0) {
if (!click || (click && !isOpen && subMenu)) {
li.addClass('open');
subMenu.addClass("visible");
}
subMenu.removeClass("hide");
}
} |
|
@Piedone I would need you to provide me with a Lombiq module that I could use for backward compatibility tests. |
|
@Skrypt could you elaborate please? I presume it should have admin features, but anything special? (BTW all of them are here: https://github.com/Lombiq/). |
|
Yes, I need one that uses admin views. I've had issues installing modules so I need one that works with the dev branch ; if you could point me on a specific one that would help. |
|
Hmm. These are some modules having admin functionality and are easy to test:
Always use the version from the dev branches. Let me know if something doesn't look right. |
|
The checkboxes are not displaying in some places after upgrading the Amaretti Theme. The reason is that we need absolutely to have the checkbox input right before it's label. In some places there is more than one input in the Thanks @Piedone I will do. |
|
Just to summarize this feature TODO. We have 2 bugs left to fix. We need to test if old modules still works. And we need to figure out if RTL still works. Will revert my current work on Content Items list header just to get a clean working version in the feature-admin branch. I'm stopping adding things in this branch. We need to ship it and I'm getting short on time. So please, feel free to help on the remaining tasks 😉 |
|
I reverted back the work on content list filters. Now the filters works as before. I also added some minor responsive styling. |
|
Culture picker is now added to the top hamburger menu. Sipke look at the background color of those icon-nav pop-up. They look different than on the Amaretti demo website. |
|
@rtpHarry please use the admin label on your issues so that we can filter those issues directly in the issues list. And thanx for your help. It's always good to have people to test/help and have second view on this. |
|
@Skrypt I don't have that level of access to the repo. I'm just a "docs team" member so I can't do tags or manage issues on the main repo |
Here is a list of the core modules that still need to have adjustments for making them display better on mobile devices :
Main tasks
Other general tasks status :
Here are the things that needs to be improved generally in all views :
.form-controlclass<div class="table-responsive">to make them overflowThe text was updated successfully, but these errors were encountered: