Skip to content

showMore/less#62

Merged
devvsakib merged 1 commit intodevvsakib:mainfrom
safil-badarudeen:readMore/less_button
Jan 19, 2023
Merged

showMore/less#62
devvsakib merged 1 commit intodevvsakib:mainfrom
safil-badarudeen:readMore/less_button

Conversation

@safil-badarudeen
Copy link
Copy Markdown
Contributor

@safil-badarudeen safil-badarudeen commented Jan 19, 2023

  • added button in the style of description itself to show more and less of description which is more than 100 letter
  • changed styling line-clamp-3 to line-clamp-11 because old styling was limiting description display functionality eventhough toggling use state works fine
  • chose line-clamp-11 because the biggest description in project in 10 line , so just chosed on line more,
  • line clamp am talking about is in Errorcard.jsx line number 56 //for future//
  • line-clamp should be increased if description not fits in 11 line
  • Fixed bug which was restricting the visibility of the Description

 - added button in the style of description itself to show more and less of description which is more than 100 letter
 - changed styling line-clamp-3 to line-clamp-11 because old styling was limiting description display functionality eventhough toggling  use state works fine
 - chose line-clamp-11 because the biggest description in project in 10 line , so just  chosed on line more,
 - line clamp am talking about is in Errorcard.jsx line number 56
         //for future//
 - line-clamp should be increased if description not fits in 11 line
@vercel
Copy link
Copy Markdown

vercel bot commented Jan 19, 2023

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

Name Status Preview Comments Updated
github-error-solve ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 19, 2023 at 5:00PM (UTC)

@safil-badarudeen
Copy link
Copy Markdown
Contributor Author

ps: only real change made in ErrorCard.jsx,
change is Error.jsx is just code formatted by prettier

Copy link
Copy Markdown
Owner

@devvsakib devvsakib left a comment

Choose a reason for hiding this comment

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

really appreciate your hard work. amazing work

@devvsakib devvsakib merged commit de0632b into devvsakib:main Jan 19, 2023
@safil-badarudeen safil-badarudeen deleted the readMore/less_button branch January 19, 2023 19:30
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