cms icon indicating copy to clipboard operation
cms copied to clipboard
verified publisher icon code100x

bug: UI of video page is cluttered

Open hkirat opened this issue 1 year ago • 8 comments

Screenshot 2024-04-02 at 4 24 58 AM

  1. Sidebar is messed up
  2. Buttons can be cleaner

hkirat avatar Apr 01 '24 22:04 hkirat

Taking this up @hkirat

Pratham271 avatar Apr 01 '24 23:04 Pratham271

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 ?.

mithun522 avatar Apr 02 '24 02:04 mithun522

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

Rimaru06 avatar Apr 02 '24 10:04 Rimaru06

@Rimaru06 I have. Thanks for asking though

Pratham271 avatar Apr 02 '24 10:04 Pratham271

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

Pratham271 avatar Apr 02 '24 10:04 Pratham271

yup can merge

hkirat avatar Apr 02 '24 10:04 hkirat

@hkirat made the pr please check

Pratham271 avatar Apr 02 '24 11:04 Pratham271

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

visheshdubey avatar Apr 13 '24 16:04 visheshdubey