Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(accessibility): replace recurrence multiselect with ncselect #5336

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JohannesGGE
Copy link
Contributor

Fix: #4618

I only replaced the deprecated NcMultiselect by NcSelect. Now the Select takes more space but when you select week for example, the left part is squeezed.

image

Signed-off-by: Johannes Merkel <mail@johannesgge.de>
@JohannesGGE JohannesGGE added bug design Related to design, interface, interaction design, UX, etc. feature: accessibility labels Jun 21, 2023
@JohannesGGE JohannesGGE self-assigned this Jun 21, 2023
@codecov
Copy link

codecov bot commented Jun 21, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (9ac0475) 22.36% compared to head (cdeba55) 22.36%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #5336      +/-   ##
============================================
- Coverage     22.36%   22.36%   -0.01%     
  Complexity      359      359              
============================================
  Files           236      236              
  Lines         11554    11555       +1     
  Branches       2250     2250              
============================================
  Hits           2584     2584              
- Misses         8970     8971       +1     
Flag Coverage Δ
javascript 14.00% <ø> (-0.01%) ⬇️
php 64.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/components/Editor/Repeat/RepeatFreqSelect.vue 0.00% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug design Related to design, interface, interaction design, UX, etc. feature: accessibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Recurrence selection is small and fiddley
1 participant