Page MenuHomePhabricator

Codex-PHP: Add support for fake buttons (link buttons)
Open, HighPublic

Description

CSS-only buttons support link buttons, but we don't have support for these in Codex-PHP. There should be a way to get Codex-PHP to output a link button.

Event Timeline

Just chiming in to say we also have a use case for this in T412087 (F68620259), in case one is needed.

Hello , I am Vanessa Bedzra
I would like to work on this task
@Catrope

Change #1257891 had a related patch set uploaded (by Vanessa_bedzra; author: Vanessa_bedzra):

[design/codex-php@main] T406372:Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1257891

Change #1257899 had a related patch set uploaded (by Vanessa_bedzra; author: Vanessa_bedzra):

[design/codex-php@main] T406372::Add snapshot test

https://gerrit.wikimedia.org/r/1257899

Change #1257899 abandoned by Vanessa_bedzra:

[design/codex-php@main] T406372::Add snapshot test

https://gerrit.wikimedia.org/r/1257899

Change #1257891 had a related patch set uploaded (by Vanessa_bedzra; author: Vanessa_bedzra):

[design/codex-php@main] Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1257891

Change #1257891 had a related patch set uploaded (by Vanessa_bedzra; author: Vanessa_bedzra):

[design/codex-php@main] T406372: Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1257891

Change #1260016 had a related patch set uploaded (by ItoroJames; author: ItoroJames):

[design/codex-php@main] Button: Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1260016

Change #1257891 had a related patch set uploaded (by Vanessa_bedzra; author: Vanessa_bedzra):

[design/codex-php@main] T406372:Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1257891

Change #1257891 had a related patch set uploaded (by Vanessa_bedzra; author: Vanessa_bedzra):

[design/codex-php@main] Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1257891

Change #1262987 had a related patch set uploaded (by EileenBlessing; author: EileenBlessing):

[design/codex-php@main] button: Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1262987

Change #1262988 had a related patch set uploaded (by EileenBlessing; author: EileenBlessing):

[design/codex-php@main] sandbox: Add examples of link buttons to ButtonExample

https://gerrit.wikimedia.org/r/1262988

LGoto subscribed.

Thanks to all the candidates who showed an interest in this project. Unfortunately, due to changing circumstances, the mentor team is no longer able to move forward with this project proposal. We appreciate all the work that’s been put into this already, and wish all the candidates the best in your future work.

Daimona removed Vanny as the assignee of this task.
Daimona added a subscriber: Vanny.

Even if no longer worked on as part of Outreachy, this remains a valid feature request, hence reopening and adjusting.

Change #1257891 merged by jenkins-bot:

[design/codex-php@main] Add support for fake buttons (link buttons)

https://gerrit.wikimedia.org/r/1257891

Change #1262987 abandoned by VolkerE:

[design/codex-php@main] button: Add support for fake buttons (link buttons)

Reason:

Thanks for the contribution! We went with I3fdae440e7807 here instead as the other contribution was slightly more advanced and originally provided earlier. Best regards

https://gerrit.wikimedia.org/r/1262987

Change #1260016 abandoned by VolkerE:

[design/codex-php@main] Button: Add support for fake buttons (link buttons)

Reason:

Thanks for the contribution. We went with I3fdae440e7807 as the patch was provided earlier. Best regards!

https://gerrit.wikimedia.org/r/1260016

Change #1262988 merged by jenkins-bot:

[design/codex-php@main] sandbox: Add additional examples of link buttons to ButtonExample

https://gerrit.wikimedia.org/r/1262988

Is there anything more to do here? It looks like the functionality was added by @Vanny and that it's now possible to provide the href key in a button's config.

@Samwilson Yes, that's right. I added support for the href key in the button configuration, so the required functionality is already in place. Unless there's another requirement I'm missing, I think this is complete.