Commit ee027ef5 authored by Jose Ivan Vargas's avatar Jose Ivan Vargas

Merge branch '1893-swap-primary-for-secondary-buttons-in-security-configure-page' into 'master'

Swap primary for secondary buttons in Security Configure page

See merge request gitlab-org/gitlab!81578
parents 7b6f2ec7 b7a31605
......@@ -31,13 +31,12 @@ export default {
const button = this.enabled
? {
text: this.$options.i18n.configureFeature,
category: 'secondary',
}
: {
text: this.$options.i18n.enableFeature,
category: 'primary',
};
button.category = 'secondary';
button.text = sprintf(button.text, { feature: this.shortName });
return button;
......@@ -126,7 +125,7 @@ export default {
v-else-if="showManageViaMr"
:feature="feature"
variant="confirm"
category="primary"
category="secondary"
class="gl-mt-5"
:data-qa-selector="`${feature.type}_mr_button`"
@error="onError"
......
......@@ -50,7 +50,7 @@ describe('FeatureCard component', () => {
expect(enableLinks.exists()).toBe(expectEnableAction);
if (expectEnableAction) {
expect(enableLinks).toHaveLength(1);
expect(enableLinks.at(0).props('category')).toBe('primary');
expect(enableLinks.at(0).props('category')).toBe('secondary');
}
const configureLinks = findConfigureLinks();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment