⚡️ Live updates comments, jobs, and issues, tagged with #beta blocker will update issues and activities on this page.

Issues

The last 100 updated issues.

Activities

The last 7 days of comments and CI jobs.

  • 🇺🇸United States chrisfromredfin Portland, Maine

    There will be more a11y fixes coming, but I aim to try to keep them as small and narrowly scoped as possible.

  • 🇺🇸United States chrisfromredfin Portland, Maine

    So given that #3 is generally the preferred behavior, the open question is still what should we do in this situation?

    (1) Go to source "D", sort by Z-A.
    (2) Jump back to source "A" which has no Z-A.
    (3) So it sets source "A" to sort by its default.

    CURRENTLY it's also setting ALL sources to sort by their defaults, too, so
    (4) Jump back to "D" and it's no longer Z-A

    Should we instead keep D's sort to Z-A, effectively meaning we're storing the active sort per source? But if that's the case, what would happen if I:

    (1) go to source "D" and sort Z-A
    (2) go to source "A" which has no Z-A so it sorts by Relevancy
    (3) change source "A" to "date"
    (3) go back to "D"
    - should be it still on Z-A? Or should it be "date" (if available) or should it be its default?

  • 🇩🇪Germany rkoller Nürnberg, Germany

    re #31 📌 Change UI for the install queue to match bulk operations Active thank you! left a few comments on gitlab that added variables to the install button states as well plus to the border of the select button states to make things more consistent. but aside that things look good. everything else should be moved to follow ups as utkarsh_33 suggested.

    re #30 📌 Change UI for the install queue to match bulk operations Active i still think it is important to be explicit about the terminology used on source type and i wouldnt consider the problem outdated but having an issue just about a single source type might be the wrong approach and there probably needs to be an issue for all source types in general. but there are a few need that need to be discussed how the bulk action bar should behave (out of the scope for this issue) if entities are selected for a source type and the person is switching source types. on admin/content the bulk action bar disappears on the new tab and if you return the previous selection is not restored. with svelte the selection is remembered, if you switch the source type after selecting entities on source type 1, there is no bar and no selection on source type 2 but if you switch back to source type 1 the selection is being restored (i m even able to switch to pages outside of project browser and if i return the filter string, filter settings and my selections are being restored?!). and within the same source type you are also able to add modules across several pages (something not possible on admin/content):

    • if you have selected several entities across several pages of a source type it is difficult to remember (in particular people with a small working memory) on which pages you have selected which entity, there is currently no way to at least reset a selection. i suppose having an interface were you get an overview of your selection and where you are able to remove individual selections would overcomplicate the UI, but having at least some kind of reset option might be helpful or even necessary?
    • about the selection being persistent. i wonder if it would be a good idea if a person is deliberately moving away from the context of project browser based pages to reset the persons selection and maybe even the string someone searched for? i imagine it might be a source for confusion and even a potential source for other problems (person unaware that a selection is still active, adding more to that, and in the end it becomes an unexpected number of installs for the person) or even be simply annoying if a selection is being persistent if someone leaves project browser pages and returns.
    • in the context of an unexpected number of modules installs, would it be reasonable to add an option to the project browser settings to enable a confirmation step for the bulk install option? that the site builder gets an overview of the things being installed and has to confirm that first? that the person has the situational awareness what is happening next (also the case that an accidental selection of the wrong module was made?)
    • in the context of the related issue it might be important to clarify that the "streams shouldnt be crossed" (i helped myself with ghostbusters). i imagine it potentially confusing if it would be possible down the line to have a "global" bulk action list for all available source types. solely terminology wise that would create tricky problems probably.
  • First commit to issue fork.
  • 🇺🇸United States chrisfromredfin Portland, Maine

    Actually, each of these bullets should be its own sub-issue. If you want to collect all modal-related a11y issues, you can do a meta and do each bullet as a child; but, we need to keep scope as SMALL as possible for issues. Moving this one back to RTBC.

  • 🇺🇸United States chrisfromredfin Portland, Maine

    Next step on this is to please read through all of Ralf's commentary in #41, identify remaining tasks, and put them IN THE ISSUE SUMMARY please!

  • 🇺🇸United States chrisfromredfin Portland, Maine

    Adding related issue here - just to note that we _may_ at some point (and it may be a future/follow-on issue) want to change the button action text based on the source plugin. So once you've picked some we may want "Apply selected recipes" for the recipe sources, and "Install selected projects" for the modules, ex.g.

  • 🇩🇪Germany rkoller Nürnberg, Germany

    thanks for the changes. over all things look better and more consistent moving the dequeue button to the secondary button style as well.thanks for that. the color for the normal and hover are correct now. the only thing missing is setting the active state when the button is pressed as well. that is still at #003ECC same as the normal state. and on a sidenote wouldnt it be the better choice to use the color variable instead of hardcoding the hex value? normal would be --color-blue-600, hover --color-blue-650, and active --color-blue-700... and the other points could be moved to follow up issues i agree.

  • @rkoller can you have a look at this again?I have addressed feedbacks that i think can be addressed as a part of this issue.

  • 🇩🇪Germany rkoller Nürnberg, Germany

    i had already commented over on slack a few days ago, but add my comment for reference here as well now as well. the only two problems i see with the current state is for one that the button style for the queue button moved from a primary button style over to a secondary button style. per se a reasonable and good move since the primary action on this page is the install button on the bulk action bar now. but it feels not correct that the dequeue button is still styled as a primary button. semantically i think it would be the right choice to style the dequeue button as a secondary button as well? that could be done within the scope of this issue. the other styling with the stylized checkboxes we've discussed might be moved to a follow up to provide visual cues aside the plain button label helping the user to distinguish the queue and dequeue buttons more easily.

    the color of the hover state has a too low color contrast against the background color of the bulk action bar, but that is an issue that core itself has with claro as well. on the thread i've suggested moving that to a follow up issue , opening an issue for claro and project browser. i'Ve also asked @mgifford about that detail, but havent had a feedback yet.

    and another problem that core has on for example admin/content as well is the fact if you have not selected anything (an entity on admin/content or a a module on admin/modules/browse) you are able to click/press the apply to selected items / install selected project button anyway and you get a no project selected error. that error should be avoided in the first place. again that could be moved to a follow up issue.

    so in summary the only thing i think from my perspecitve that would definitley make sense within the scope of this issue is changing the styling of the dequeue buttons to secondary buttons as well. what do others think?

    ps: oh and i just noticed the secondary button is missing the state style changes (hover and active). the blue remains the same here? https://www.figma.com/design/OqWgzAluHtsOd5uwm1lubFeH/%F0%9F%92%A7-Drupa...

  • 🇮🇳India Kanchan Bhogade

    Trying to apply MR! 634 on Drupal 10 but getting an error
    also In summary MR shows "Open Failed Pipeline"

    Attaching screenshot for reference

  • 🇩🇪Germany rkoller Nürnberg, Germany

    thanks for the changes... the dialog modal getting the initial focus looks good... but i've noticed several more problems in the context of the a11y within the dialog modal. so far i have only tested it with modules that havent had an image carousel, plus i havent taken a closer look with voiceover active neither (see and hear dialog.mp4 and voiceover.mp4).

    • with the dialog modal being the first element in focus it feels sort of odd to have the close button as the second in the tabindex? would it be possible and be better if the close button would become the last in the tabindex in the dialog modal?
    • the back and forth buttons of the image carousel in the dialog modal are missing a visible focus outline (SC2.4.7).
    • The labels for the next button with "slide right" is not clear (why not use something like "next image" and "previous image") in particular due to the fact that all the images in the carousel are labeled as decorational. with only two images you only have a single button cuz the other is disabled. with more than two images you then have a slide left and slide right buttons but no content you the user are sliding inbetween.
    • the disabled button is not reflected in the aural interface and that way things might become confusing for screenreader users. using aria-disabled="true" instead of disabled would make the disabled button tabable and labeled as dimmed/disabled in for example voicover
    • the tab order is odd. as demonstrated in voiceover.mp4, if the focus is on the slide right button, you press return and the button turns disabled and you press shift-tab to get back to the slide left button you get to the learn more button instead?!
    • in dialog.mp4 and voiceover.mp4 i manage each time to get with the focus outside the dialog modal into the background of the modal which should not happen.

    i set the issue back to needs work. the other option might be to move some of those points to follow up issues to keep the scope for this issue, but due to the fact that this issue is about the a11y of the dialog modal i set it back to needs work for now

  • 🇮🇳India narendraR Jaipur, India

    Tested manually and focus is now set on modal. Moving it to RTBC.

  • I am not sure if color of Install selected projects and Select/Unselect is correct or not.

    So i was trying to make this inline with the comment mentioned in #5 📌 Change UI for the install queue to match bulk operations Active by @chrisfromredfin. Not sure if that's not what we want.It would be helpful if someone clearly describes what behaviour we want for the select and deselect buttons.
    Or i think we can take that as a follow-up because this issue is majorly fixing the install button behaviour to match the bulk action.

  • Automatically closed - issue fixed for 2 weeks with no activity.

  • 🇮🇳India narendraR Jaipur, India

    Below are some findings:

    • Still shows 1 item selected
    • I think on clicking Install selected projects when no project is selected, it should scroll to top where the error message is shown
    • Cursor : pointer in select button
    • I am not sure if color of Install selected projects and Select/Unselect is correct or not.
  • 🇮🇳India Kanchan Bhogade

    Trying to apply MR!634 but getting an error

    Attaching screenshot for reference

  • Addressed all the feedbacks and tests are also passing.Ready for review.

Production build 0.71.5 2024