Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

Commit

Permalink
Remove todo not on o-form demo markup.
Browse files Browse the repository at this point in the history
It may be an issue if `o-forms` decides to style an `input` with
`.o-forms-input--select` or a `select` with `.o-forms-input--text`.
We don't think this is likely to be a problem and may be caught
with visual regression testing.
  • Loading branch information
notlee committed Jun 25, 2021
1 parent 536ce3b commit c20963c
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion demos/src/static/static.mustache
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
</span>
{{! o-forms styles for select input needed for the core experience }}
{{! o-forms styles for text input needed for the enhances JS experience }}
{{! todo: document this as a valid/supported usecase of o-forms? make autocomplete depend on o-forms and toggle? }}
<span class="o-forms-input o-forms-input--text o-forms-input--select">
<div data-o-component="o-autocomplete" class="o-autocomplete">
<select id="my-autocomplete">
Expand Down

0 comments on commit c20963c

Please sign in to comment.