Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.0] Category Blog menu item type parameters layout #34081

Closed
joomdonation opened this issue May 21, 2021 · 6 comments
Closed

[4.0] Category Blog menu item type parameters layout #34081

joomdonation opened this issue May 21, 2021 · 6 comments

Comments

@joomdonation
Copy link
Contributor

Steps to reproduce the issue

Create a menu item to link to Category Blog menu item type, look at Blog Layout tab to change menu item parameters.

Expected result

The layout looks good.

Actual result

The layout looks mess. See this attached image
category-blog-menu-option

System information (as much as possible)

Joomla 4 nightly build

Additional comments

Could one of our contributors with frontend take a look at this and make sure it is sorted before RC release?

@chmst
Copy link
Contributor

chmst commented May 21, 2021

see also #33480


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34081.

@eopws
Copy link

eopws commented May 21, 2021

Is it ok to wrap the fields to another line when not having enough container width?

Or something else?

@chmst
Copy link
Contributor

chmst commented May 21, 2021

@eopws I think that this is ok.

My impression is, that the problem happens with an odd number of elements.

@eopws
Copy link

eopws commented May 22, 2021

@chmst

My impression is, that the problem happens with an odd number of elements.

Actually no. It happens when the elements are too big. Count of elements doesn't matter

odd
even

Or I didn't understand you right

@eopws
Copy link

eopws commented May 22, 2021

What about addin' a horizontal scrollbar?

@richard67
Copy link
Member

Closing as having a pull request. Please test #34145 . Thanks in advance.

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

No branches or pull requests

5 participants