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

GridView: Fit image to grid item without crop #1692

Conversation

TomaszLukawskiSam
Copy link
Contributor

@TomaszLukawskiSam TomaszLukawskiSam commented Oct 8, 2021

New css class has been added to Gridview widget
The ui-gridview-image-icon fits image size to the grid item container.

The GridView widget detects icons and adds ui-gridview-image-icon to image.

obraz

Signed-off-by: Tomasz Lukawski [email protected]

@TomaszLukawskiSam TomaszLukawskiSam marked this pull request as draft October 8, 2021 19:07
@TomaszLukawskiSam TomaszLukawskiSam force-pushed the issue-change-image-size-in-gridview branch 3 times, most recently from d997b7c to 8bd9374 Compare October 11, 2021 17:37
@TomaszLukawskiSam TomaszLukawskiSam marked this pull request as ready for review October 11, 2021 17:42
@@ -164,6 +164,18 @@ tau-gridview {
}
}

&.ui-gridview-image-icon .ui-gridview-item img,
Copy link
Collaborator

Choose a reason for hiding this comment

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

If the girdview image is mixed with general image and icon
and I don't know which image will be on the grid , how about it?
I think in that case I can't use the "ui-gridview-image-icon".

@TomaszLukawskiSam TomaszLukawskiSam force-pushed the issue-change-image-size-in-gridview branch from 8bd9374 to 27cb01e Compare October 12, 2021 20:51
@TomaszLukawskiSam TomaszLukawskiSam force-pushed the issue-change-image-size-in-gridview branch from 5a69f6b to 81c8483 Compare October 12, 2021 21:44
@TomaszLukawskiSam TomaszLukawskiSam force-pushed the issue-change-image-size-in-gridview branch 2 times, most recently from b7a3fff to e6708a9 Compare October 13, 2021 09:57
Signed-off-by: Tomasz Lukawski <[email protected]>
@TomaszLukawskiSam TomaszLukawskiSam force-pushed the issue-change-image-size-in-gridview branch from e6708a9 to c2c6f4c Compare October 13, 2021 10:17
Copy link
Collaborator

@singa2000 singa2000 left a comment

Choose a reason for hiding this comment

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

Fix the image in the gridview

@TomaszLukawskiSam TomaszLukawskiSam merged commit 633273f into Samsung:tau_1.2 Oct 13, 2021
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