Skip to content

Conversation

@tblivet
Copy link
Collaborator

@tblivet tblivet commented Jul 13, 2023

Questions Answers
Description? Add the CSS part for backward compatibility
Type? new feature
BC breaks? --
Deprecations? --
Fixed ticket? --
Sponsor company @PrestaShopCorp
How to test? This file can be used for testing : example/bootstrap-4.html

@tblivet tblivet marked this pull request as ready for review July 17, 2023 16:54
@tblivet tblivet requested a review from ga-devfront July 17, 2023 16:54
Copy link
Collaborator

@ga-devfront ga-devfront left a comment

Choose a reason for hiding this comment

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

Really nice work, after checking all the things it's all good for me, I got just one suggestion.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it would me good to make the same change from example/bootstrap-4.html to see more easily the difference between us.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, I will add these changes on this page too 👍

@tblivet tblivet requested a review from ga-devfront July 18, 2023 14:36
Copy link
Collaborator

@ga-devfront ga-devfront left a comment

Choose a reason for hiding this comment

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

Perfect thank you for your amazing work !

</script>
</body>

</html> No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

required new line at end of file

@ga-devfront ga-devfront merged commit a47dc8c into main Jul 19, 2023
@ga-devfront ga-devfront deleted the css-compatibility branch July 19, 2023 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants