execute `classRemove` on `autocompleteresponses` only

Created on 26 February 2021, over 4 years ago
Updated 15 December 2023, over 1 year ago

classRemove should be ONLY called on `autocompleteresponses`

https://git.drupalcode.org/project/drupal/-/blob/9.2.x/core/themes/claro...

it should not hide the `loading` when the response hasn't returned.
Second, the CORE rely on `CompositionEvent` event for IME, it will stop before KEY STROKES (isComposing)


(haven't followed these thing a long time. the style looks like a bug?)

🐛 Bug report
Status

Postponed: needs info

Version

11.0 🔥

Component
Claro 

Last updated 5 days ago

Created by

🇭🇰Hong Kong droplet

Live updates comments and jobs are added and updated live.
  • JavaScript

    Affects the content, performance, or handling of Javascript.

  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇮🇳India Akhil Babu Chengannur

    The given issue summary is not clear. The code shared https://git.drupalcode.org/project/drupal/-/blob/9.2.x/core/themes/claro... no longer exists in drupal 10/11. I also searched core directory for 'autocompleteresponses', but couldn't find any matching results.

    Additionally, I cross-checked the alignment of the loading indicator in both D9 and D10 sites (Looks misaligned in the screenshot) and found that it's correctly aligned in both versions.
    D10

    D9

    Please confirm if the issue is still valid

  • Status changed to Closed: outdated 15 days ago
  • 🇺🇸United States smustgrave

    Since there's been no follow up going to close out, if still an issue in D11 please re-open

    Thanks!

Production build 0.71.5 2024