Commit 0b82f6e2 authored by Nicolò Maria Mezzopera's avatar Nicolò Maria Mezzopera

Merge branch '300435-add-label' into 'master'

Resolve "Add GlToggle label in `app/assets/javascripts/sidebar/components/subscriptions/subscriptions.vue`"

See merge request gitlab-org/gitlab!54548
parents 3dcdcfb5 2753854d
......@@ -122,6 +122,8 @@ export default {
:value="subscribed"
class="hide-collapsed"
data-testid="subscription-toggle"
:label="__('Notifications')"
label-position="hidden"
@change="toggleSubscription"
/>
</div>
......
---
title: Add GlToggle label in sidebar subscription toggle
merge_request: 54548
author: Yogi (@yo)
type: changed
......@@ -84,6 +84,15 @@ describe('Subscriptions', () => {
spy.mockRestore();
});
it('has visually hidden label', () => {
wrapper = mountComponent();
expect(findToggleButton().props()).toMatchObject({
label: 'Notifications',
labelPosition: 'hidden',
});
});
describe('given project emails are disabled', () => {
const subscribeDisabledDescription = 'Notifications have been disabled';
......
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