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

Added object position for images #39930

Closed
wants to merge 1 commit into from

Conversation

IarinaX
Copy link

@IarinaX IarinaX commented Apr 29, 2024

Description

Motivation & Context

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would change existing functionality)

This pull request addresses the issue #39196 by introducing support for the object-position property within Bootstrap's utility classes. The absence of this property limited developers' ability to precisely control the positioning of images within their projects. To remedy this, we have implemented a set of utility classes that enable developers to specify the position of images with greater flexibility and granularity.

Changes:

Added new utility classes for object-position property.
Defined utility classes for centering, top-left, top-right, top-center, bottom-left, bottom-right, and bottom-center positioning options.
Ensured compatibility with Bootstrap's design principles and naming conventions.
Testing:

Conducted thorough testing to verify the functionality and compatibility of the new utility classes across different browsers and devices.
Tested various positioning scenarios to ensure consistent behavior and visual presentation.
Impact:

Enhances the flexibility and usability of Bootstrap's image handling capabilities.
Empowers developers to create visually appealing and well-structured layouts with ease.
Improves the overall user experience for developers working with Bootstrap-based projects.

  • I have read the contributing guidelines
  • My code follows the code style of the project (using npm run lint)
  • My change introduces changes to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed

Live previews

Related issues

Closes #39196

@IarinaX IarinaX requested a review from a team as a code owner April 29, 2024 11:11
@IarinaX IarinaX closed this Apr 29, 2024
@IarinaX IarinaX reopened this Apr 29, 2024
@julien-deramond julien-deramond marked this pull request as draft April 29, 2024 18:16
Copy link
Member

@julien-deramond julien-deramond left a comment

Choose a reason for hiding this comment

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

Thanks for submitting a PR, @IarinaX!

As mentioned in this comment, we haven't yet discussed the addition of this feature to Bootstrap.

As we aren't sure that we will embed this feature, and because I respect your time and effort (I don't want you to work for nothing), I'll be closing this PR for now due to some lint issues and the need for additional documentation.

If we decide to move forward with this feature, I'll be sure to reopen this PR so we can pick up from where we left off.

Thanks again for your contribution!

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

Successfully merging this pull request may close these issues.

Object position for images
2 participants