Skip to content

Conversation

@Rajveer07
Copy link
Contributor

No description provided.

Copy link
Contributor

@PradyumnaKrishna PradyumnaKrishna left a comment

Choose a reason for hiding this comment

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

Please format the files as well using npm run format.


<div class="gallery flex flex-col lg:w-1/2 m-auto items-center">
<div class="overflow-y-auto">
<div class="flex flex-col gap-4 sm:flex-row sm:flex-wrap md:flex-col lg:flex-row lg:flex-wrap xl:flex-col">
Copy link
Contributor

Choose a reason for hiding this comment

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

This could be simple

Suggested change
<div class="flex flex-col gap-4 sm:flex-row sm:flex-wrap md:flex-col lg:flex-row lg:flex-wrap xl:flex-col">
<div class="flex flex-wrap flex-col lg:flex-row">

<div class="overflow-y-auto">
<div class="flex flex-col gap-4 sm:flex-row sm:flex-wrap md:flex-col lg:flex-row lg:flex-wrap xl:flex-col">
<div style="display: flex;">
<div class="w-full overflow-hidden rounded-lg shadow-lg mb-4 flex justify-center items-center">
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess this might work, but try to make it look good.

Suggested change
<div class="w-full overflow-hidden rounded-lg shadow-lg mb-4 flex justify-center items-center">
<div class="basis-1/2 grow-0 max-h-[400px] overflow-hidden rounded-lg shadow-lg mb-4 flex justify-center items-center">

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am trying

@PradyumnaKrishna
Copy link
Contributor

If you wan't able to complete this don't worry. I didn't love it much, we will figure something else out. We can avoid adding it.

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