AutoSuggest icon indicating copy to clipboard operation
AutoSuggest copied to clipboard

Update AutoSuggest.js

Open mhassman opened this issue 3 years ago • 2 comments

getComputedStyle(element, 'line-height') was returning 'normal' thus NaN and placing the .top position incorrectly at zero. Fixed by defaulting to font-height if line-height isn't numeric.

mhassman avatar May 19 '21 02:05 mhassman

@mhassman getComputedStyle method is costly and you are calling it twice, please store it in a variable and reuse it

avcs06 avatar Jul 19 '21 13:07 avcs06

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Dec 13 '21 14:12 sonarqubecloud[bot]