Skip to content

Update buttons to use Pajamas::ButtonComponent

Ahmed Hemdan requested to merge ahemdan/pajamas-migration-day-buttons-2 into master

What does this MR do and why?

This merge request updates the buttons in the following files to use Pajamas::ButtonComponent.

File Issue
app/views/devise/shared/_omniauth_box.html.haml #417814 (closed)
app/views/clusters/clusters/cloud_providers/_cloud_provider_button.html.haml #417812 (closed)

I used the instructions provided in this issue, but feedback is always welcome!

Screenshots

Before After
Screenshot_2023-07-14_at_5.47.31_PM Screenshot_2023-07-14_at_5.44.39_PM
Screenshot_2023-07-14_at_5.50.18_PM Screenshot_2023-07-14_at_5.52.30_PM

Please note: one of the buttons was removed from this merge request because it is not trivial and require further testing and styling considerations, please see these two comments (1, 2) for more information.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Ahmed Hemdan

Merge request reports