Ref T4420. I think the border is good enough without this spinny thing, and it needs a lot of code.
Details
Details
- Reviewers
btrahan chad - Maniphest Tasks
- T4420: Modernize typeahead datasources
- Commits
- Restricted Diffusion Commit
rP6696d2e85aa4: Remove spinner GIF for loading tokenizers
Used typeahead.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- tokbeh
- Lint
Lint Passed - Unit
No Test Coverage
Event Timeline
Comment Actions
We have the luxury of usually be local/intranet installs for most people. Curious how this will do on SaaS, but haven't felt we needed it on secure.
Comment Actions
Yeah, I think we could find other display options to make it a little more prominent without needing like 50 lines of code too. We add/remove a class even without this code, and could do various things to make the effect more visible.