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

[FEATURE] gray badge #125

Open
wants to merge 9 commits into
base: feature/show-maxitems-in-pagemodule
Choose a base branch
from

Conversation

KlickUndKlar
Copy link

Hi @IchHabRecht

I made the Style of the badge and fixed some issues with the drag and drop area.

I tried to place the badge without an absolute position, but i had to change too much on the wrapping element styles.
So i went back to the absolute positioning. Because of that the badge is placed in front of the column title.

Bildschirm­foto 2023-02-09 um 09 43 10

Depending of the length of the badge, i gave a «padding-left» to the column title. Because of that i set a limit to the maxitem variable. If there is a value over 99, i write a string «+99» to the badge.

Bildschirm­foto 2023-02-09 um 09 36 01

Bildschirm­foto 2023-02-09 um 09 42 51

Greets Ben

@KlickUndKlar KlickUndKlar changed the title gray badge [FEATURE] gray badge Feb 9, 2023
@IchHabRecht
Copy link
Owner

Hi @IchHabRecht

Thank you very much for your effort. I will start a review to ask some minor questions.

@@ -1,34 +1,37 @@
@charset "UTF-8";

.t3-cd-badge-container {
position: relative;
.t3js-page-column .t3-cd-badge-container[data-maxitems="0"] .t3-page-ce .t3js-page-new-ce,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I understand the CSS correctly for a full column there wouldn't be any badge anymore. But one of the main purposes is to tell the editor that there is a limit and it is reached.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, the Line 3-12 just removes the «new content-element» Button not the badge.
Line 8-12 is a fix, to prevent the shifting of the Drop area if the button is hidden.

The badge is always visible. We just hide it if there is no maxitems property in the backendlayout configuration.

}

.t3-cd-badge {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you tested the layout in all TYPO3 versions yet?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i say it in German – "verdammt – da war noch was 😳"

I keep you updated after testing

@KlickUndKlar
Copy link
Author

Hi @IchHabRecht we tested, the branch in a T3 10, 11 and 12.
As you can see, we added an additional CSS File for T3 12 because there are some Major changes in the Backend Layout.
I'm not sure, if there is a better Solution than a Viewhelper to get the Major Version of the CMS. Let me know if we have to change things.

@IchHabRecht
Copy link
Owner

Hi @KlickUndKlar

Thank you very much for your help. I will review the changes as soon as I find some time.

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

Successfully merging this pull request may close these issues.

3 participants