fix: extended gap in title and description

[Anmol-Baranwal]

Fixes Issue

Resolves #1050

Description

• I have fixed the following bug in title on popup:

Previous	Current

Note to reviewers

@rupali-codes @ujjawaltyagii
I need your help, so the issue is that the description in the Card is slightly longer, and I have properly debug the code on various components.

This image shows the problem

So, on changing

- className="w-full cursor-pointer transition-all duration-100 ease-in hover:scale-[1.02] md:w-72"
+ className="cursor-pointer transition-all duration-100 ease-in hover:scale-[1.02] md:w-50"

in CardListItem.tsx

The gap is fixed as shown below:

But this solution doesn't meet my intended goal. Could you provide any ideas or approaches to address this issue?

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
linkshub	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 1, 2023 0:16am

[github-actions]

Thank you, Anmol-Baranwal, for creating this pull request and contributing to LinksHub! 💗

The maintainers will review this Pull Request and provide feedback as soon as possible! 😇
We appreciate your patience and contribution, Keep up the great work! 😀

[CBID2]

I think this helps

[rupali-codes]

@Anmol-Baranwal hi, so for the cards like this, i guess it would be better if we can align the text to left for all cards? what do you you think?

[Anmol-Baranwal]

@Anmol-Baranwal hi, so for the cards like this, i guess it would be better if we can align the text to left for all cards? what do you you think?

I have already described the issue in the pr comment on what I reached after debugging, so are you suggesting to align it on left but it would still not fix the issue, because i literally added 'hey' in between the gap while testing. Please clarify further.

[Anmol-Baranwal]

@rupali-codes
Please provide clarification.

[rupali-codes]

@rupali-codes Please provide clarification.

I guess, it would be better if we add a gap between the words than using text-justify because its adding unnecessary extra space, or let me know what do you think about it

[Anmol-Baranwal]

@rupali-codes Please provide clarification.

I guess, it would be better if we add a gap between the words than using text-justify because its adding unnecessary extra space, or let me know what do you think about it

I will complete the remaining PRs after tomorrow's exam, by the 24th.

[Anmol-Baranwal]

@rupali-codes Please provide clarification.

I guess, it would be better if we add a gap between the words than using text-justify because its adding unnecessary extra space, or let me know what do you think about it

There is no text-justify used in codebase, and it seems that the styles are overlapping each other. As a result, I'm unable to make any significant changes to resolve this issue. This problem may be attributed to the fixed size of the card.

Can you please look again.

[rupali-codes]

@rupali-codes Please provide clarification.

I guess, it would be better if we add a gap between the words than using text-justify because its adding unnecessary extra space, or let me know what do you think about it

There is no text-justify used in codebase, and it seems that the styles are overlapping each other. As a result, I'm unable to make any significant changes to resolve this issue. This problem may be attributed to the fixed size of the card.

Can you please look again.

Its coming from here actually,

[CBID2]

Looking good

[ujjawaltyagii]

Looking good

Ig he is not done with the final changes as i can still see the unusual gaps in resource description

[CBID2]

Looking good

Ig he is not done with the final changes as i can still see the unusual gaps in resource description

Hence why I chose the general comment option 😊

[Anmol-Baranwal]

@rupali-codes
On removing it as per your suggestion, the design looks like this. Please let me know if this design is acceptable or if the current one works better.

Without Removing

Removing

[rupali-codes]

the second one looks much better to me

[Anmol-Baranwal]

the second one looks much better to me

I have done the changes as per your preference, the problem was styles in globals.css was at higher priority, so I changed the tag from p to div

[rupali-codes]

the second one looks much better to me

I have done the changes as per your preference, the problem was styles in globals.css was at higher priority, so I changed the tag from p to div

Thanks, looks nice to me