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.
Description
Details
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T406373 Use fake/link buttons instead of regular buttons in places where that would be better | |||
| Open | None | T406372 Codex-PHP: Add support for fake buttons (link buttons) |
Event Timeline
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)
Change #1257899 had a related patch set uploaded (by Vanessa_bedzra; author: Vanessa_bedzra):
[design/codex-php@main] T406372::Add snapshot test
Change #1257899 abandoned by Vanessa_bedzra:
[design/codex-php@main] T406372::Add snapshot test
hi @Catrope , https://gerrit.wikimedia.org/r/c/design/codex-php/+/1257891 PR made, pending review
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)
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)
Change #1260016 had a related patch set uploaded (by ItoroJames; author: ItoroJames):
[design/codex-php@main] Button: Add support for fake buttons (link buttons)
Hello, I have added a patch:
https://gerrit.wikimedia.org/r/c/design/codex-php/+/1260016
Please review
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)
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)
Change #1262987 had a related patch set uploaded (by EileenBlessing; author: EileenBlessing):
[design/codex-php@main] button: Add support for fake buttons (link buttons)
Change #1262988 had a related patch set uploaded (by EileenBlessing; author: EileenBlessing):
[design/codex-php@main] sandbox: Add examples of link buttons to ButtonExample
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.
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)
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
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!
Change #1262988 merged by jenkins-bot:
[design/codex-php@main] sandbox: Add additional examples of link buttons to ButtonExample
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.