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

hero-unit: fix redesign issues #5409

Merged
merged 1 commit into from
Jan 31, 2024
Merged

hero-unit: fix redesign issues #5409

merged 1 commit into from
Jan 31, 2024

Conversation

hom3mad3
Copy link
Contributor

@hom3mad3 hom3mad3 commented Jan 22, 2024

hero-unit: redesign fixes

hero-unit-fix

possible duplicate of: #5406 and #5404

Tasks

  • Changelog

Copy link
Contributor

@philli-m philli-m left a comment

Choose a reason for hiding this comment

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

coolio! nice catch on the width issue, that passed me by, just a note about class names and the linting, but soon linting will be mostly done! (hopefully)

@philli-m
Copy link
Contributor

Oh and no double up with the tabs PR by the way, thought I wrote that somewhere but it seems lost now?!

@hom3mad3 hom3mad3 force-pushed the mr-2024-01-project-hero-fixes branch 3 times, most recently from 1c6e9da to 9477c80 Compare January 30, 2024 22:32
@hom3mad3 hom3mad3 requested a review from philli-m January 30, 2024 22:33
@philli-m philli-m force-pushed the mr-2024-01-project-hero-fixes branch from 9477c80 to d38319a Compare January 31, 2024 15:28
@@ -1,10 +1,16 @@
.follow__container > span {
width: 100%;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

@hom3mad3 this span is the data widget from the template ta hence no class and as it is an inline element it had a limited width (i think anyway) so it was also limiting the nested elements

Copy link
Contributor

@philli-m philli-m left a comment

Choose a reason for hiding this comment

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

perfect! :) will approve and should auto merge after CI

@philli-m philli-m merged commit 2f3bc71 into dev Jan 31, 2024
3 checks passed
@philli-m philli-m deleted the mr-2024-01-project-hero-fixes branch January 31, 2024 15:40
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.

2 participants