Problem/Motivation

QA testing of Olivero across browser - Samsung Internet. Please test the latest major version.

This particular issue is more of a visual style against a lot of supported browsers that we might not use on a day-to-day basis

Breaking up the parent issue: QA testing of Olivero across multiple browsers #3173877: [meta] QA testing of Olivero across multiple browsers. This was discussed in Slack on April 26.

Testing script: https://docs.google.com/document/d/10UVObMlSMQHpFPwOvrFWDtXP2kzl1ZEx6PlU...

Steps to complete

  1. Read testing script above
  2. See example video and issue at #3210745: QA testing of Olivero - Desktop browser: Google Chrome
  3. Test browser and record video
  4. Post video

Comments

volkswagenchick created an issue. See original summary.

mherchel’s picture

Issue tags: +Olivero stable blocker
mherchel’s picture

Issue summary: View changes
mherchel’s picture

Thanks for the review @KondratievaS!

Addressing some the bugs

1) pager buttons - It's a feature not a bug :) The pager buttons will wrap when the viewport becomes too narrow
2.1) Webform radios not centered - This should be solved by setting the visually-hidden class on the label, which needs to be done in the webform module
2.2) This does look like a bug.
3) I'm not worried about horizontal scrolling caused by table layouts (this is in the document)
4) I'm not worried about table header sticky interactions. That's outside the scope of the theme. (the table header page isn't slated to be reviewed in the document)
5) I'm not worried about dialog sizes. This is set by jQueryUI and is external to the theme.

Is there a video showing the walkthrough?

mherchel’s picture

Status: Active » Needs review
Issue tags: -Needs issue summary update

Performed video of walkthrough as I don't see one in comment #4:

https://www.youtube.com/watch?v=Vaf28brWYvs

proeung’s picture

@mherchel Thanks for putting together a video walkthrough of the Samsung Internet mobile browser testing! Everything looks good, however, I'm not seeing bug 2.2 that @KondratievaS mentioned in comment #4 in your walkthrough.

Do we have an issue already created for the unexpected lines when a form field has been added? If so, we can use that to track the bug and close out this testing issue, since every else looks to be working fine?

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

Drupal 9.1.10 (June 4, 2021) and Drupal 9.2.10 (November 24, 2021) were the last bugfix releases of those minor version series. Drupal 9 bug reports should be targeted for the 9.3.x-dev branch from now on, and new development or disruptive changes should 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.

mherchel’s picture

Status: Needs review » Fixed
StatusFileSize
new45.93 KB

Not sure if this is something that we changed, or Samsung Internet changed, but the bug in 2.2 is now not occurring (see screenshot). Marking this as fixed per #7.

Status: Fixed » Closed (fixed)

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