cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

[CAL-1612] Dark mode bugs availability

Open Udit-takkar opened this issue 1 year ago β€’ 10 comments

Found a bug? Please fill out the sections below. πŸ‘

Issue Summary

  1. set as default on hover
Screenshot 2023-05-03 at 1 12 28 AM
  1. delete icon on hover
Screenshot 2023-05-03 at 1 12 52 AM

should be like this Screenshot 2023-05-03 at 1 18 13 AM

  1. Delete icon on hover should be red
Screenshot 2023-05-03 at 1 13 43 AM

Steps to Reproduce

  1. go to http://localhost:3000/availability
  2. Open any availability item

CAL-1612

Udit-takkar avatar May 02 '23 19:05 Udit-takkar

@MaheshB77 you can work on this

Udit-takkar avatar May 02 '23 19:05 Udit-takkar

If available, can I work on this issue?

Adithya2907 avatar May 02 '23 19:05 Adithya2907

@MaheshB77 you can work on this

Sure @Udit-takkar Please assign this to me, will fix this

MaheshB77 avatar May 02 '23 19:05 MaheshB77

@Adithya2907 you can look for any other issue or just use app.cal.com and look for any responsive or dark mode ui bugs and create an issue for it

Udit-takkar avatar May 02 '23 20:05 Udit-takkar

@Udit-takkar sure, I'll try looking to other issues to fix. Thank you!

Adithya2907 avatar May 02 '23 20:05 Adithya2907

I believe , this issue is already solved image

skushagra9 avatar May 04 '23 17:05 skushagra9

@skushagra9 no that's a different page. go to /availability and open any availability item

Udit-takkar avatar May 04 '23 17:05 Udit-takkar

@Udit-takkar see this image

skushagra9 avatar May 04 '23 17:05 skushagra9

Hi!! is this issue still open?

Ksh1t1j avatar May 08 '23 00:05 Ksh1t1j

@Udit-takkar for the second issue the colour seems to be that way for a disabled button. If there are multiple schedules, the delete button is active and works as intended. So my question is this, would it be better to have a different style for an inactive button or is the current one good enough?

Saga-sanga avatar May 09 '23 16:05 Saga-sanga

So I tried running an instance locally and the "set to default" under /availabilty seems to have been fixed but is not visible in production.

https://github.com/calcom/cal.com/assets/70441807/57c6a8d4-a4bd-462b-9d69-11c0d1b9b95c

RayAsh37 avatar May 10 '23 11:05 RayAsh37

@RayAsh37 ok someone must have merged the PR

Udit-takkar avatar May 10 '23 11:05 Udit-takkar