⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@smistad
Copy link

@smistad smistad commented Apr 19, 2021

Fixes issue #196
Detect if browser is on mobile device, and if so skip the appending of autocomplete text.
Detecting mobile browser taken from here: https://stackoverflow.com/questions/11381673/detecting-a-mobile-browser

Should probably just hide the key guide in the search dialog as well on mobile since they are not relevant...?

@codecov
Copy link

codecov bot commented Apr 19, 2021

Codecov Report

Merging #197 (1dd69b8) into master (3a29473) will decrease coverage by 0.10%.
The diff coverage is 53.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #197      +/-   ##
==========================================
- Coverage   98.28%   98.18%   -0.11%     
==========================================
  Files          27       27              
  Lines        6646     6655       +9     
  Branches       44       46       +2     
==========================================
+ Hits         6532     6534       +2     
- Misses        114      121       +7     
Impacted Files Coverage Δ
documentation/search.js 98.07% <53.33%> (-1.93%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a29473...1dd69b8. Read the comment docs.

friendlyanon pushed a commit to friendlyanon/m.css that referenced this pull request Jul 15, 2021
@mosra
Copy link
Owner

mosra commented Jan 6, 2022

As mentioned in #196, this got solved in a different way in b598223, with an attempt to restore the autocompletion functionality stashed in #213.

@mosra mosra closed this Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants