Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

fix: Truncate to work well with w-full or w-division. #155

Merged
merged 1 commit into from
Oct 17, 2022
Merged

fix: Truncate to work well with w-full or w-division. #155

merged 1 commit into from
Oct 17, 2022

Conversation

xiCO2k
Copy link
Collaborator

@xiCO2k xiCO2k commented Oct 9, 2022

This PR adds a bug fix when using truncate with w-full or w-1/2.

render(<<<HTML
    <div class="mx-2 my-1">
        <div class="w-full truncate">Lorem ipsum dolor, sit amet consectetur adipisicing elit. Sunt illo et nisi omnis porro at, mollitia harum quas esse, aperiam dolorem ab recusandae fugiat nesciunt doloribus rem eaque nostrum itaque.</div>
    </div>
HTML);

Before

image

After

image

@xiCO2k xiCO2k requested a review from nunomaduro October 9, 2022 10:45
@nunomaduro
Copy link
Owner

@xiCO2k Can you test this pull request, with any potential output effected on Laravel Framework? If is working good, you can merge and release.

@xiCO2k
Copy link
Collaborator Author

xiCO2k commented Oct 17, 2022

Sure thing, will do later today.

@xiCO2k xiCO2k merged commit 24ee35e into nunomaduro:master Oct 17, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@nunomaduro nunomaduro Awaiting requested review from nunomaduro

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xiCO2k @nunomaduro