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

Add Support my work section #169

Merged
merged 2 commits into from
Mar 5, 2023
Merged

Add Support my work section #169

merged 2 commits into from
Mar 5, 2023

Conversation

szepeviktor
Copy link
Owner

@szepeviktor szepeviktor commented Mar 2, 2023

👀
https://github.com/szepeviktor/phpstan-wordpress/blob/szepeviktor-patch-1/README.md#support-my-work
@adamlaki There seems to be a color difference and this heart does not beat :(

What if we add the same ids/classes -> then GitHub's CSS will work on our svg too?

@szepeviktor szepeviktor self-assigned this Mar 2, 2023
@szepeviktor szepeviktor merged commit a49f83b into master Mar 5, 2023
@szepeviktor szepeviktor deleted the szepeviktor-patch-1 branch March 5, 2023 10:16
@adamlaki
Copy link

adamlaki commented Mar 5, 2023

Viktor,

you asked this via e-mail; why didn't you continue the discussion over there?

Please, don't give half the information. If a color does not match, what color is it?

What kind of animation do you want, just on the hover state (like the button)?

@szepeviktor
Copy link
Owner Author

Please, don't give half the information. If a color does not match, what color is it?

I'm sorry.
This PR fixes the color. szepeviktor/.github#1
Could you approve it?

What kind of animation do you want, just on the hover state (like the button)?

Yes, hover-only animation.
Is that possible in SVG?
This button is used in README-s.

@adamlaki
Copy link

adamlaki commented Mar 5, 2023

I've approved it, looks good.

Yes, in theory, it is possible. I will make the addition soon.

@adamlaki
Copy link

adamlaki commented Mar 5, 2023

I don't understand where the color diff is coming from; it is correct in the file that I sent you.

@szepeviktor
Copy link
Owner Author

szepeviktor commented Mar 5, 2023

I've added some SVG features and optimization.

    <rect width="92" height="28" ...
    <path fill="#BF3989" ...
    <text ...

Add messed up the color code ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants