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

Adapt CSS classes for badges to BS5 #313

Merged

Conversation

richard67
Copy link

Pull Request for Issue #311 .

Summary of Changes

Adapt to J4's change to BS5 for the badges: Use classes bg-... as suggested by @ceford , thanks for reporting and suggesting.

Other glitches from the BS5 change I haven't seen yet.

Testing Instructions

Install Patchtester 4 on a current 4.0-dev branch or latest 4.0 nightly build or the last beta, which all have already the change to BS5 included.

Fetch the pull requests and apply one patch to see the different colours.

Result:

2021-02-10_03

Apply the changes from this PR and refresh the page,

Result:

2021-02-10_04

@richard67 richard67 mentioned this pull request Feb 10, 2021
@ceford
Copy link

ceford commented Feb 10, 2021

I had a problem applying the patch - Eclipes IDE would not show my local source file in the Patch dialog, maybe because I installed it rather than forked/cloned it. So I applied the differences by text edit. It works fine. Is there a way to mark Pass here? I normally only use the Joomla tracker.

@richard67
Copy link
Author

No, we don't have the tracker for this project, we only use comment, like you did now.

I also had to apply the change manually.

All fine.

@richard67
Copy link
Author

@ceford What you can do is to close your issue #311 as you have tested that this PR fixes it.

@particthistle
Copy link

Thanks for the fix @richard67. Nearly logged another duplicate while Philip and I were onboarding GSoC students yesterday and we spotted the issue.

@roland-d roland-d merged commit 05edf1b into joomla-extensions:master Mar 3, 2021
@roland-d
Copy link
Collaborator

roland-d commented Mar 3, 2021

Thank you @richard67 @ceford

@roland-d roland-d added this to the 4.1.0 milestone Mar 3, 2021
@richard67 richard67 deleted the master-fix-badges-for-bs5 branch March 3, 2021 19:59
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.

4 participants