Skip to content
This repository has been archived by the owner on Jan 25, 2021. It is now read-only.

Module chrome without card header - edit icon overlaps dropdown buttons and texts #87

Open
chmst opened this issue Sep 17, 2020 · 19 comments

Comments

@chmst
Copy link
Collaborator

chmst commented Sep 17, 2020

What needs to be fixed

Impossible to open a submenu in this case:

grafik

If a module is displayed without title AND the user may edit, then the area for the edit button must have mor height

@richard67
Copy link
Member

Oops.

@chmst chmst changed the title Metismenu vertical Metismenu vertical - module chrome issue Sep 17, 2020
@chmst
Copy link
Collaborator Author

chmst commented Sep 17, 2020

I would prefer that the edit symbol is always visible , not only on hover.

@richard67
Copy link
Member

Ping @infograf768 : Could you check @chmst 's previous comment?

@infograf768
Copy link
Member

I am now preparing an eclipse project of this Cassiopea stuff.
I may look at things tomorrow.

@infograf768
Copy link
Member

I don't remember if we had a discussion on this when it was implemented.
I sort of think that it should behave like the article or contact and be present when user can edit.
But, in some cases, it would break the display so much that I have mixed feelings.

the best solution may be to display the tooltip up, at the top limit of the module. Not sure we can. And, in any case not sure either our frontend tooltips are a11y.

@infograf768
Copy link
Member

For now the tooltip placement is "auto"
If we modify to "top", we will get (mobile screencapture)
Screen Shot 2020-09-17 at 17 50 27
It's automatically contained in the window btw.

.tooltip({container: $('.jmodedit').parent(), html: true, placement: 'auto'});

@richard67
Copy link
Member

As far as I understand, the problem is not the tool tip but the icon. It overlaps the metismenu toggle when using a metismenu in the sidebar, e.g. by moding the "Main Menu Blog" module to there.

@infograf768
Copy link
Member

That's a totally other issue.

@infograf768
Copy link
Member

infograf768 commented Sep 18, 2020

I nevertheless suggest to change tip placement as the issue of the tip covering up the first link in other types of modules is present. (In 3.x it is set to top by default)
Also, we have a huge margin-left for the <ul> in the modules in general we should get rid of and I could not find where that comes from.

Screen Shot 2020-09-18 at 08 02 01

I could solve it by using (for LTR). But it has to be more specific as this mod-list is used in other modules and breaks them.

.mod-list {
    margin-left: -2rem;
}

To solve the overlap problem, some changes in the css will make it

.btn.jmodedit
{
  z-index: 1001;
  display: block;
  position: absolute;
  top: -1rem; // proposal
  right: -1rem; // proposal
}
html[dir=rtl] .btn.jmodedit
{
    right: auto;
    left: -1rem; //proposal
}

This gives (modified css a bit vs the animated gif below)
Screen Shot 2020-09-18 at 09 38 18

and use
editmenumodule

I remark that the submenu is hardly differentiated from its parent.

Note

With metismenu we have lost the edit menu item possibility.

@chmst
Copy link
Collaborator Author

chmst commented Sep 18, 2020

The edit buttons are added via frontendediting.js. This makes changes difficult. And in my opinion it is not userfreindly that the edit button is visible only with hover.

A good an simple and a11y solution could be (in core as this is rendering modules): If editing a module is possible: Add a new line on top of the module, containig the edit icon + edit label as in articles and contacts.

If this is not possible, adding some more padding-top to a metismenu in the sidebars could do it too.

@chmst
Copy link
Collaborator Author

chmst commented Sep 18, 2020

grafik

Same issue here on small screen: It is not possible to open the menu - the edit button is nearyl invisible on the dark background.

@infograf768
Copy link
Member

If this is not possible, adding some more padding-top to a metismenu in the sidebars could do it too.

I am afraid it is the only way to go. But it should be done only when frontendediting is set.
I saw no way for now to render an edit icon without the js. And no way to integrate menu item edit when using metismenu.

@infograf768
Copy link
Member

@chmst
Only way I found is to totally get rid of the js, modifying the layout and also the chromes.

Proof of concept (did not correct all css)

Screen Shot 2020-09-19 at 10 21 43

@chmst
Copy link
Collaborator Author

chmst commented Sep 20, 2020

If we add a div (100% width) bove the module content, it looks like this:
grafik

@infograf768
Copy link
Member

infograf768 commented Sep 29, 2020

We could get rid of the part of the js concerning the edit module and modify frontediting_modules.php. This would work with the card chrome in the side positions but would fail with the system chromes.

@chmst
Copy link
Collaborator Author

chmst commented Oct 14, 2020

We must get rid of the jQuery part at least in the frontend, as this is not accessible. It is not about chromes but about the layout in cms. Added label joomla-cms

@chmst chmst changed the title Metismenu vertical - module chrome issue Module chrome without card header - edit icon overlaps dropdown buttons and texts Oct 24, 2020
@chmst
Copy link
Collaborator Author

chmst commented Oct 24, 2020

It is not only vertical menu but every module without card header. Must be solved in joomla cms and imho it is a release blocker. But this is soething I cannot decide.

@chmst
Copy link
Collaborator Author

chmst commented Nov 5, 2020

I am not sure if it is cassiopeia or core but terrible wrong

@chmst
Copy link
Collaborator Author

chmst commented Nov 5, 2020

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants