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

Feat: add new design for categories page #2195

Merged
merged 24 commits into from
Jan 13, 2024
Merged

Conversation

gakshita
Copy link
Contributor

@gakshita gakshita commented Nov 25, 2023

fixes #2177

Changes proposed

New design for category page implemented

rupali-codes and others added 13 commits November 2, 2023 23:20
* feat: Add Swift Sub Category

* Fix small correction

* Update database/languages/swift.json

Co-authored-by: Christine Belzie <[email protected]>

---------

Co-authored-by: Christine Belzie <[email protected]>
* [ADDED]: BlockChain Category

* Changes done

* Update index.ts

* Update index.ts

* Changes made

* Update CategoryDescriptions.ts
* feat: add iconsax icon

* chore: update description

Co-authored-by: Rupali Haldiya <[email protected]>

---------

Co-authored-by: Anmol Baranwal <[email protected]>
Co-authored-by: Rupali Haldiya <[email protected]>
* Added 2 new podcasts

* Added Overpowered.AI podcast

* Added new link to podcasts

* fix: json structure

* chore: update description

Co-authored-by: Christine Belzie <[email protected]>

* chore: remove whitespace

Co-authored-by: Aftab Rehan <[email protected]>

---------

Co-authored-by: Anmol Baranwal <[email protected]>
Co-authored-by: Christine Belzie <[email protected]>
Co-authored-by: Aftab Rehan <[email protected]>
* UI libraries in fronted added

* added description for the UI libraries

* description update for materailui , antdesign, chakraui

* removed os tutorials
Copy link

vercel bot commented Nov 25, 2023

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 Jan 13, 2024 9:03am

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you, gakshita, 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! 😀

Copy link
Collaborator

@CBID2 CBID2 left a comment

Choose a reason for hiding this comment

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

@Anmol-Baranwal Anmol-Baranwal linked an issue Nov 26, 2023 that may be closed by this pull request
@Anmol-Baranwal Anmol-Baranwal linked an issue Nov 26, 2023 that may be closed by this pull request
@gakshita
Copy link
Contributor Author

The page looks good @gakshita, but can you revert the commits that don't pertain to the issue that your PR solves?

Have removed the unecessary commits

Copy link
Collaborator

@aftabrehan aftabrehan left a comment

Choose a reason for hiding this comment

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

Excellent PR, @gakshita 💪 🚀 💯

Thanks for the contribution. It just needs minor adjustments and then we're good to go.

However, I have only reviewed the code yet. I'll proceed with UI testing and provide my feedback in a while. Feel free to ask if you have any questions or want clarification on something.

assets/icons/AddIcon.tsx Outdated Show resolved Hide resolved
pages/[category]/index.tsx Outdated Show resolved Hide resolved
pages/[category]/index.tsx Outdated Show resolved Hide resolved
pages/[category]/index.tsx Outdated Show resolved Hide resolved
pages/[category]/index.tsx Outdated Show resolved Hide resolved
pages/[category]/index.tsx Outdated Show resolved Hide resolved
components/Breadcrumps/index.tsx Outdated Show resolved Hide resolved
@aftabrehan
Copy link
Collaborator

  1. Could you please align this section to the top?
Screenshot 2023-11-27 at 7 36 09 PM
  1. Could you please make this element clickable (button/link), and redirect users to https://github.com/rupali-codes/LinksHub/issues/new/choose. We'll edit this link in the future. BTW, I think a component for this is already created. Here it's used.
Screenshot 2023-11-27 at 7 38 17 PM
  1. The titles look too bold. Can you please make it thin?
Screenshot 2023-11-27 at 7 42 55 PM
  1. Oops, it's not ready for the light mode. We have a light design mode available in Figma.
Screenshot 2023-11-27 at 7 44 30 PM
  1. I guess it's not responsive, is it?
Screenshot 2023-11-27 at 7 46 38 PM

No need to stress! If you ever need assistance, feel free to reach out. We're here to support and help you on your learning journey. 🚀

@gakshita
Copy link
Contributor Author

@aftabrehan @rupali-codes @CBID2 all changes have been fixed from my side!

Copy link
Collaborator

@aftabrehan aftabrehan left a comment

Choose a reason for hiding this comment

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

Thanks @gakshita! The code looks much better now.

However, here are a few suggestions from my side.

pages/[category]/index.tsx Outdated Show resolved Hide resolved
tailwind.config.js Outdated Show resolved Hide resolved
@aftabrehan
Copy link
Collaborator

It all looks super awesome 🙌. Great work @gakshita. Please, help to adjust these two things as well, and then we're good to merge this PR.

  1. Can you please add the background on the parent element?
Screenshot 2023-11-30 at 6 06 59 PM
  1. Can you please use a full width for the tile on the mobile screen to cover extra space?
Screenshot 2023-11-30 at 6 14 45 PM

@Anmol-Baranwal
Copy link
Collaborator

@gakshita
Are you facing any problems regarding the changes? Please let me know; I will gladly help you.

@gakshita
Copy link
Contributor Author

gakshita commented Dec 3, 2023

Couldn't find item description data i the object, rest changes have been pushed.

@aftabrehan
Copy link
Collaborator

Couldn't find item description data i the object, rest changes have been pushed.

@gakshita, Please, check this for reference.

const category = router.asPath
const categoryName = category.split('/')[1].split('-').join(' ')
const subcategoryName = category?.split('/')[2]?.split('-').join(' ')
const searchQuery = router.query.query?.toString() || ''
let cleanedCategory = ''
cleanedCategory = (subcategoryName || categoryName)
.replaceAll(regEx, ' ')
.replaceAll('search query ', '')
const description = categoryDescriptions[searchQuery || subcategoryName] || ''
const isResourceSelected = isValidResource(searchQuery || subcategoryName)

@CBID2
Copy link
Collaborator

CBID2 commented Dec 17, 2023

Any updates @gakshita? 😁

@CBID2 CBID2 added goal: new-design Design related issues priority: high Making completely new feature labels Dec 22, 2023
@rupali-codes
Copy link
Owner

@gakshita you still working on it?

Copy link

vercel bot commented Jan 13, 2024

Someone is attempting to deploy a commit to a Personal Account owned by @rupali-codes on Vercel.

@rupali-codes first needs to authorize it.

@rupali-codes rupali-codes merged commit aac8277 into rupali-codes:dev Jan 13, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
goal: new-design Design related issues priority: high Making completely new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: add new design for categories page