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.

Sign up for GitHub

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

Jump to bottom

The size of the image card should be fixed. on the home page. #6654

Closed
Sanjaybhattwebkul opened this issue Aug 23, 2022 · 1 comment
Closed

The size of the image card should be fixed. on the home page. #6654

Sanjaybhattwebkul opened this issue Aug 23, 2022 · 1 comment
Assignees
@papnoisanjeev
Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working. UI UI package related issue.
Milestone
1.4.4

Comments

@Sanjaybhattwebkul
Copy link
Contributor

Bagisto Version: Master

Steps To reproduce--

1. Visi the shop in the default theme
2. View the product which has no image.

Expected result: The size of the image card should be fixed. on the home page.

Actual result: The size of the image card is not fixed if there is no image for any product. on the home page. Please find the screenshot Here
@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Something isn't working. UI UI package related issue. labels Aug 23, 2022
@Sanjaybhattwebkul Sanjaybhattwebkul added this to the 1.4.4 milestone Aug 23, 2022
@papnoisanjeev papnoisanjeev mentioned this issue Aug 23, 2022
Merged
@papnoisanjeev papnoisanjeev added the Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. label Aug 23, 2022
jitendra-webkul added a commit that referenced this issue Aug 24, 2022
@jitendra-webkul
Merge pull request #6655 from papnoisanjeev/fixed-image-size-default-…
b0e4d0d 
…theme

Image height fixed for default theme #6654
@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Fixed When bug issue is fixed. and removed Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. labels Aug 25, 2022
@Sanjaybhattwebkul
Copy link
Contributor Author

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@papnoisanjeev papnoisanjeev

Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working. UI UI package related issue.
Projects
None yet
Milestone
1.4.4
Development

No branches or pull requests
2 participants
@papnoisanjeev @Sanjaybhattwebkul