search

code100x / cms

https://app.100xdevs.com/
867 stars 1.2k forks source link

bug: UI of video page is cluttered #313

Closed hkirat closed 2 days ago

hkirat commented 5 months ago
Screenshot 2024-04-02 at 4 24 58 AM
  1. Sidebar is messed up
  2. Buttons can be cleaner
Pratham271 commented 5 months ago

Taking this up @hkirat

mithun522 commented 5 months ago

Hey guys I am sorry to comment my doubt here. I dont know where i could go and ask thats why i am commenting it out here. Can only the members who have purchased the courses can contribute to this ?.

Rimaru06 commented 5 months ago

@Pratham271 have you already fix this if not i would also like to fix this

Pratham271 commented 5 months ago

@Rimaru06 I have. Thanks for asking though

Pratham271 commented 5 months ago
Screenshot 2024-04-02 at 3 49 54 PM Screenshot 2024-04-02 at 3 49 57 PM Screenshot 2024-04-02 at 3 50 03 PM

@hkirat bhaiya looks good to me. If there is any suggestion from your side I can make that change before generating a pr

hkirat commented 5 months ago

yup can merge

Pratham271 commented 5 months ago

@hkirat made the pr please check

visheshdubey commented 4 months ago

Hi @hkirat, I have made a figma layout to de-clutter the video page, I have also considered adding some other UI/UX fixes. Please have a look. If it looks good to you, I can start working on this!

  1. I have moved back button from sidebar to the breadcrumbs
  2. Instead of a fluid layout in main section, added a contained layout for better responsiveness on large screens.
  3. Improved AppBar, and search input.
  4. Suggestion : To use shadcn scrollbars
  5. Other UI fixes.

image 6 image 7 image 8

Thanks