Problem/Motivation

In IE11 (I know, but we're stuck with it until Drupal 10 at the earliest 😢), when keys are pressed while the mobile search input is focused, no letters are *visually* added to the input, the cursor just stays blinking at the beginning of the input. Note that the input is actually receiving text, typing something and pressing return will trigger a search for what was typed, but it's not visually appearing in the input when typed

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bnjmnm created an issue. See original summary.

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

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.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

Issue tags: -JavaScript +CSS

Verified this is still an issue. This is a CSS bug though, and can be fixed by adjusting the padding on the input. We need to ensure that there's no visual regressions, though.

kostyashupenko’s picture

Status: Active » Needs review
FileSize
999 bytes
xjm’s picture

Issue tags: +Internet Explorer 11

Unfortunately "invisible input" is bad enough that we cannot quite demote this to minor like the other IE 11 issues. If it were anything other than IE 11 I would promote the issue. Part of me still feels like I should for the sake of 9.4 sites' users encountering this.

As a workaround for that, I think that we might not want to make Olivero the default theme of 9.4 if this bug is outstanding. I would make an exception in this case and allow it to be the default theme of 10.0 only despite the policy otherwise that 9.4 and 10.0 have parity.

mherchel’s picture

I think that we might not want to make Olivero the default theme of 9.4 if this bug is outstanding

This issue is a relatively simple CSS fix. We should be able to get this committed if it's a stable blocker for 9.3/9.4

mherchel’s picture

Issue tags: +Olivero stable blocker

Discussed with with several core committers on August 20th, and we agreed that this should be a stable blocker.

mherchel’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
136.14 KB
83.96 KB
92.32 KB
104.37 KB

This looks perfect. I tested this out on IE11 and verified that it works as expected.

I also tested out Chrome, Firefox, and Safari for regressions, and there were none.

  • lauriii committed 838b903 on 9.3.x
    Issue #3190537 by kostyashupenko, mherchel, bnjmnm, xjm: Mobile search...

  • lauriii committed bfada41 on 9.2.x
    Issue #3190537 by kostyashupenko, mherchel, bnjmnm, xjm: Mobile search...
lauriii’s picture

Title: Mobile search input in IE11 does not visually respond to keypress » Mobile search input on tablet width in IE11 does not visually respond to keypress
Version: 9.3.x-dev » 9.2.x-dev
Status: Reviewed & tested by the community » Fixed

Committed 838b903 and pushed to 9.3.x. Also cherry-picked to 9.2.x because Olivero is experimental. Thanks!

Status: Fixed » Closed (fixed)

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