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

[NV-2195] 🐛 Bug Report: Visual bug in Add filter button icon #3313

Closed
2 tasks done
michaldziuba03 opened this issue May 1, 2023 · 24 comments · Fixed by #4151
Closed
2 tasks done

[NV-2195] 🐛 Bug Report: Visual bug in Add filter button icon #3313

michaldziuba03 opened this issue May 1, 2023 · 24 comments · Fixed by #4151
Labels
bug Something isn't working Hacktoberfest help wanted Extra attention is needed linear

Comments

@michaldziuba03
Copy link
Contributor

michaldziuba03 commented May 1, 2023

📜 Description

It's small visual bug. Icon doesn't change color on theme change.

👟 Reproduction steps

  1. Navigate to template editor
    image

  2. Set theme to light
    image

👍 Expected behavior

Icon should change color to keep contrast after theme change.

👎 Actual Behavior with Screenshots

Look at Add filter button:

dark
light

As we can see - the icon doesn't change color when changing the theme.

📃 Provide any additional context for the Bug.

No response

👀 Have you spent some time to check if this bug has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

Are you willing to submit PR?

Yes I am willing to submit a PR!

NV-2195

@davidsoderberg davidsoderberg changed the title 🐛 Bug Report: Visual bug in Add filter button icon [NV-2195] 🐛 Bug Report: Visual bug in Add filter button icon May 1, 2023
@himanshu1221
Copy link

Can I work on this issue?

@scopsy
Copy link
Contributor

scopsy commented May 3, 2023

Sure @himanshu1221, assigning you 🚀

@scopsy scopsy removed the triage label May 3, 2023
@scopsy scopsy added the bug Something isn't working label May 3, 2023
@himanshu1221
Copy link

hey @scopsy can you guide me on how I can setup this up in gitpod

@scopsy
Copy link
Contributor

scopsy commented May 16, 2023

Haven't used GitPod in a while now :( But you will only need to run the @novu/api and @novu/web projects to see this.

@gitstart
Copy link
Contributor

@scopsy is this issue still available for pickup?

@himanshu1221
Copy link

@scopsy is this issue still available for pickup?

I am working on it @gitstart

@gitstart
Copy link
Contributor

@scopsy is this issue still available for pickup?

I am working on it @gitstart

Alright

@gitstart
Copy link
Contributor

Are you still working on this? @himanshu1221

@jainpawan21
Copy link
Member

@himanshu1221 unassigned you from the issue
Waiting for someone else to pick up

@gitstart
Copy link
Contributor

@jainpawan21 we would like to work on this.

@harshhere905
Copy link

hey can i take the issue. I would like to work on it

@snehpr966
Copy link

Would like to fix it

@victoriatrinita
Copy link

I would like to work on this issue

@vishalkhoje
Copy link
Contributor

Kindly assign this issue to my name. I am presently engaged in addressing a related matter, as it pertains not only to a filter but also involves various other issues concerning light theme icons. I am committed to rectifying all of these issues comprehensively.

@vishalkhoje
Copy link
Contributor

Hello @michaldziuba03 @jainpawan21

Link to PR: #4356

I've included screenshots and a video to provide additional context and details regarding the fix for this visual bug. Please take a look.

Thank you! 🙏🏼

@ASCE-D
Copy link

ASCE-D commented Oct 3, 2023

Hey Devs I would like to contribute can you please assign me this issue

@sidonweb
Copy link

sidonweb commented Oct 4, 2023

Hello! I'd like to contribute to this issue. Can you assign me this?

@davidsoderberg
Copy link
Contributor

Hi @harshhere905 are you ready to work on this one?

@vishalkhoje
Copy link
Contributor

Hi @davidsoderberg ,
If possible can you able to consider or please check my PR: #4356
I raised it PR yesterday.
Thank you!

@harshhere905
Copy link

yes @davidsoderberg please assign me this task

@davidsoderberg
Copy link
Contributor

@harshhere905 you are now assigned to this task :)

@harshhere905
Copy link

@davidsoderberg please unassign me this task

@vishalkhoje
Copy link
Contributor

vishalkhoje commented Oct 8, 2023

Please assign me @davidsoderberg
I have already raised the PR for this few days ago.
I am also working on another issue that will also, I will try to finish today.
Thanks

@davidsoderberg
Copy link
Contributor

@snehpr966 are you ready to work on this?

@jainpawan21 jainpawan21 added the help wanted Extra attention is needed label Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Hacktoberfest help wanted Extra attention is needed linear
Projects
None yet