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

Hacked a way to link dashed line to Get Started #191

Merged
merged 3 commits into from
Sep 15, 2023
Merged

Hacked a way to link dashed line to Get Started #191

merged 3 commits into from
Sep 15, 2023

Conversation

meyerweb
Copy link
Member

@meyerweb meyerweb commented Sep 11, 2023

Closes #188

This does cause the button to not look like a button on the Get Started page, but like all the other links. This was forced by the existing CSS, and may be desirable anyway, since it keeps the Call-To-Action button from looking like a Call To Action on the page it’s trying to get you to visit. If it must continue to look like a button, even on its own page, significant rewriting of the template script and CSS will probably be needed.


Site preview: https://igalia.github.io/wpewebkit.org/issue-188/

@meyerweb meyerweb requested a review from bkardell as a code owner September 11, 2023 16:36
@csaavedra
Copy link
Member

Thanks for working on this! Personally this works for me, but I think others should chime in on whether the results are ok before we merge.

@meyerweb meyerweb enabled auto-merge (squash) September 13, 2023 01:50
Copy link
Member

@mariospr mariospr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is good enough, and probably worth getting it now than waiting for too long again, so merging.

Thanks!

@meyerweb meyerweb merged commit 41f49d7 into master Sep 15, 2023
1 check passed
@meyerweb meyerweb deleted the issue-188 branch September 15, 2023 09:39
mariospr added a commit that referenced this pull request Sep 15, 2023
Closes #188

This is basically the fix implemented in [1], which was rendered
useless after PR #189 [2] got merged, so this is a quick fix.

[1] #191
[2] #189

Site preview: https://igalia.github.io/wpewebkit.org/issue-188-v2
mariospr added a commit that referenced this pull request Sep 15, 2023
Closes #188

This is basically the fix implemented in [1], which was rendered
useless after PR #189 [2] got merged, so this is a quick fix.

[1] #191
[2] #189

Site preview: https://igalia.github.io/wpewebkit.org/issue-188-v2
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.

The broken line on the header doesn't point to "Get started" when visiting the "Get started" section
3 participants