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
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | safari.png | 104.37 KB | mherchel |
| #8 | firefox.png | 92.32 KB | mherchel |
| #8 | chrome.png | 83.96 KB | mherchel |
| #8 | ie11.png | 136.14 KB | mherchel |
| #4 | 3190537-3.patch | 999 bytes | kostyashupenko |
Comments
Comment #3
mherchelVerified 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.
Comment #4
kostyashupenkoComment #5
xjmUnfortunately "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.
Comment #6
mherchelThis 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
Comment #7
mherchelDiscussed with with several core committers on August 20th, and we agreed that this should be a stable blocker.
Comment #8
mherchelThis 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.
Comment #11
lauriiiCommitted 838b903 and pushed to 9.3.x. Also cherry-picked to 9.2.x because Olivero is experimental. Thanks!