sButtons / sbuttons

:bulb: Simple buttons you can use easily for your next project.
https://sbuttons.netlify.app/
MIT License
402 stars 558 forks source link

Spotlight button's text color in dark mode #1609

Closed shahednasser closed 2 years ago

shahednasser commented 2 years ago

In dark mode, Spotlight's text color should be white before the animation as well.

github-actions[bot] commented 2 years ago

Hi, @shahednasser, Thanks for opening an issue :blue_heart: your issue will be seen by maintainers soon. Contributors :people_holding_hands: like you make the open source community :earth_africa: such an amazing place to learn :book: , inspire :angel:, and create :art: . We will investigate :eyes: and get back to you as soon as possible :+1: . Just make sure you have given us sufficient information :information_source:. By that time enjoy this joke :point_down: , hope you like it :smile:

UDP is better in the COVID era since it avoids unnecessary handshakes.

Use this action on your projects. Use memes on issues action instead.

JLambertazzo commented 2 years ago

Hi! I'd like to work on this issue, can you please assign me?

Grandstaff79 commented 2 years ago

Hello, I would like to work on this issue as well, would you assign me ?

Rhut-virani commented 2 years ago

Hello, Can I get assigned to work on this issue?

Edit:

a similar issue is there for Glow button also.

ej-sanmartin commented 2 years ago

Since @crisboxing11 has successfully merged a change regarding this bug, issue should be closed! Did anyone check if this change, however, doesn't affect light mode?

ej-sanmartin commented 2 years ago

Read the commits / merge history wrong, guess now would be a good time to test it out again

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

shahednasser commented 2 years ago

This issue is still not resolved in case anyone is interested in working on it.

JLambertazzo commented 2 years ago

I'll take it, can you assign me?

shahednasser commented 2 years ago

Sure @JLambertazzo

JLambertazzo commented 2 years ago

I can't get the changes to display on the example site. Setting the button class to dark-mode or it's parent to have data-theme="dark" manually makes it work as expected but by default in dark mode it does not change.

CodEZ47 commented 2 years ago

This issue is still not resolved in case anyone is interested in working on it.

Can I work on this issue?

shahednasser commented 2 years ago

@JLambertazzo can you submit a PR so that I can see exactly what's going on?

JLambertazzo commented 2 years ago

Created the pull request, let me know what you think

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

Poirei commented 2 years ago

Can I join to work on this issue?

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

shahednasser commented 2 years ago

@Poirei are you still interested in working on this?

lam-brian commented 2 years ago

Hi, can I work on this issue? Also, quick question: Would you want white text on all the spotlight buttons except for the white buttons? Otherwise when you hover it, the text would blend in with the background color.

shahednasser commented 2 years ago

@lam-brian the text in dark mode should be white before the animation. As for after the animation you should just fix the white button to be another color

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

Dustpie commented 2 years ago

Can I work on this issue please?

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

github-actions[bot] commented 2 years ago

Hi, @shahednasser, Thanks for opening an issue :blue_heart: your issue will be seen by maintainers soon. Contributors :people_holding_hands: like you make the open source community :earth_africa: such an amazing place to learn :book: , inspire :angel:, and create :art: . We will investigate :eyes: and get back to you as soon as possible :+1: . Just make sure you have given us sufficient information :information_source:. By that time enjoy this joke :point_down: , hope you like it :smile:

UDP is better in the COVID era since it avoids unnecessary handshakes.

Use this action on your projects. Use memes on issues action instead.

shahednasser commented 2 years ago

Hello @Dustpie are you still interested in working on this?

rushithakkar873 commented 2 years ago

Hello @shahednasser, If no one is working on this issue then can I work on it?

shahednasser commented 2 years ago

Hello @Ruc08 sure I'll assign you

rushithakkar873 commented 2 years ago

Thank you, @shahednasser! I'm starting to work on it now :)

rushithakkar873 commented 2 years ago

Hey @shahednasser,

I'm getting some kind of prompt when I run npm start in terminal which I hadn't faced before and I'm not able see anything on localhost. It would be great if you can help me.

Screenshot 2022-07-01 at 6 48 11 PM
shahednasser commented 2 years ago

@Ruc08 basically this command watch any changes you make to the .less files and compiles them. If you want to run the website, it's under the website directory.

Please check the contributing guidelines for more details

rushithakkar873 commented 2 years ago

Hey @shahednasser, I made pull request just now. Please review it and if there's no problem than kindly merge it :)

rushithakkar873 commented 2 years ago

Hey @shahednasser, Good Afternoon!

Have you seen the PR or not?

rushithakkar873 commented 2 years ago

Hello @shahednasser, If you get some time than please look at the PR request which I made last week.

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

shahednasser commented 2 years ago

@Ruc08 are you still working on this?

rushithakkar873 commented 2 years ago

@shahednasser Yep! As I don't know LESS so need some time to learn it and to do require changes. I hope that you won't mind to give me some time to learn about LESS, Right?

Youngtechie commented 2 years ago

Hi @shahednasser just going through this issue now, which i noticed some changes i can make to resolve it. Can I be assign?

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.

rupnarayanb commented 2 years ago

Hi @shahednasser Is it still open ? I have solved it, can you please assign the issue to me ?

github-actions[bot] commented 2 years ago

@shahednasser reporting a stale issue.