Comments

mherchel created an issue. See original summary.

mgifford’s picture

Should we do the same with assistive technology like NVDA & VoiceOver?

mherchel’s picture

Title: [meta] QA testing across multiple browsers » [meta] QA testing of Olivero across multiple browsers
Project: Olivero » Drupal core
Version: 8.x-1.x-dev » 9.1.x-dev
Component: Code » Olivero theme
volkswagenchick’s picture

Issue summary: View changes

Added child issues for individual browser testing. WIll add issues as they are created, thanks.

volkswagenchick’s picture

Issue summary: View changes
thejimbirch’s picture

Still need to determine what functions will need to be tested.

Will there be different things to test on mobile and desktop browsers?

mherchel’s picture

Issue tags: +Olivero stable blocker
thejimbirch’s picture

Thread from Drupal slack: https://drupal.slack.com/archives/CJT807H7T/p1632752290045700?thread_ts=...

Andy
Just went digging into the bartick, seven, claro, and umami issue queues and none of them have issues specifically for browser testing the project as a whole.

Was there a specific maintainer that said we needed whole-project browser testing or is that an issue we put on ourselves?

https://www.drupal.org/node/3079238
Contributors manually test significant changes in multiple browsers to watch for regressions, but this is somewhat based on the contributors available and the particular issue. We do not yet have full automated browser testing for our supported browsers.
This sounds like browser testing is required for each individual feature, but not necessarily as a whole.

If a maintainer did request this, could they advise or provide guidance? TIA!

mherchel’s picture

Issue summary: View changes

Adding testing script

mherchel’s picture

updating summary

proeung’s picture

catch’s picture

Just went digging into the bartick, seven, claro, and umami issue queues and none of them have issues specifically for browser testing the project as a whole.

Bartik and Seven were added during the major development cycle (i.e. before minor releases and experimental), and then there were issues like #2152519: [meta] Make sure Drupal 8 looks good and works right on browsers (both mobile+desktop) to retrospectively test them before release.

Umami is perma-experimental so there hasn't been a step where it's marked stable.

Claro isn't stable yet.

So Olivero is the first new core theme to go from experimental to stable in any meaningful way, meaning we don't have any clear precedents.

My personal feeling here is that there's been a lot of informal testing of Olivero on multiple browsers, and some of that could have been documented on this issue earlier in the process, to save having to retest everything at the end - even if that might not have meant a formal script run-through probably would have been enough. Having said that can't have too much testing.

mherchel’s picture

gábor hojtsy’s picture

Parenting to making it the default now.

andypost’s picture

Version: 9.1.x-dev » 9.3.x-dev

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

sk33lz’s picture

StatusFileSize
new18.32 KB

I found an issue on the Olivero test site where the progress percentage displays directly next to the progress message within AJAX progress and AJAX small progress with no spacing between the text and the percentage. I initially noticed this while testing the progress bars doing QA on Safari, but I also verified the same issue exists on all browsers I've tested. This includes browsers marked as fully tested with their QA issues closed, so posting here. This particular issue exists in the latest versions of Safari, Chrome, Firefox, IE 11, Edge, and Brave, so I would have to say it's across the board. See the attached screenshot taken on a desktop screen at 1080P.

gábor hojtsy’s picture

Issue tags: +Portland2022
mherchel’s picture

lauriii’s picture

Status: Active » Fixed

All issues from the roadmap have been closed!

lauriii’s picture

phjou’s picture

I added a comment on the Firefox one earlier, it is very difficult to read the search placeholder for me:
https://www.drupal.org/project/drupal/issues/3210746
Should I create a specific issue?

catch’s picture

@phjou yes please create a new issue!

Status: Fixed » Closed (fixed)

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