Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add autocomplete component #24

Merged
merged 1 commit into from
Oct 1, 2018

Conversation

mdeniz
Copy link

@mdeniz mdeniz commented Oct 1, 2018

Have an autocomplete input field component to be based on

@mdeniz mdeniz requested a review from dmarcoux October 1, 2018 09:56
@hennevogel
Copy link
Member

hennevogel commented Oct 1, 2018

Deploy preview for obs-patterns ready!

Built with commit cc2c5cc

https://deploy-preview-24--obs-patterns.netlify.com

Copy link
Contributor

@dmarcoux dmarcoux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor changes, otherwise 👍

layout: post
title: Autocomplete
category: components
description: Whenever a field have the ability of autocomplete its value we should add a hint in form of placeholder and
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whenever a field has the ability to autocomplete its value, (new comma) we should add (...)

%span.input-group-text#search-icon-id
%i.fa.fa-search
%i.fas.fa-spinner.fa-spin.d-none
%input.form-control#input-id{type: 'text', placeholder: 'Type to autocomplete...'}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing spaces inside the brackets (so it should be like line 12). This would trigger Rubocop in OBS.

Have an autocomplete input field component to be based on
@mdeniz
Copy link
Author

mdeniz commented Oct 1, 2018

Please re-review it @dmarcoux 👍

@dmarcoux dmarcoux merged commit ecfc500 into openSUSE:master Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants