Skip to content

Extend hero box#1058

Merged
dgarcia360 merged 3 commits intoscylladb:masterfrom
dgarcia360:extend-hero-box
Apr 5, 2024
Merged

Extend hero box#1058
dgarcia360 merged 3 commits intoscylladb:masterfrom
dgarcia360:extend-hero-box

Conversation

@dgarcia360
Copy link
Copy Markdown
Collaborator

@dgarcia360 dgarcia360 commented Apr 2, 2024

Related issue scylladb/scylladb-docs-homepage#11

Edits the hero box extension to make the call to action more visible.

How to test

  1. Build the docs.

  2. Open http://127.0.0.1:5500/examples/hero-box/#hero-box-with-everything

  3. Check how the new button will look like:

    image

@dgarcia360 dgarcia360 marked this pull request as ready for review April 2, 2024 11:48
@dgarcia360 dgarcia360 requested a review from annastuchlik April 2, 2024 11:48
Copy link
Copy Markdown
Collaborator

@annastuchlik annastuchlik left a comment

Choose a reason for hiding this comment

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

LGTM

@annastuchlik
Copy link
Copy Markdown
Collaborator

@tzach FYI This theme update will make it possible to update the Get Started button on the landing page. It applies the solution you've approved.
To make any future updates, we'll need to work with a graphic designer.

@dgarcia360 dgarcia360 merged commit e1095b4 into scylladb:master Apr 5, 2024
@tzach
Copy link
Copy Markdown
Collaborator

tzach commented Apr 5, 2024

Is this change will change all sites or just scylladb-docs-homepage ?

@annastuchlik
Copy link
Copy Markdown
Collaborator

@tzach This PR adds functionality to the Sphinx theme, so it will be applied to all projects. It's configurable, so customization is per project, and you can have different buttons in different projects, as documented here: https://sphinx-theme.scylladb.com/master/examples/hero-box.html

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.

3 participants