Skip to content
This repository was archived by the owner on Aug 7, 2020. It is now read-only.

feat(oui-inline-adder): add inline-adder component #304

Merged
merged 1 commit into from
Nov 5, 2018

Conversation

AxelPeter
Copy link
Contributor

@AxelPeter AxelPeter commented Oct 31, 2018

refactor(oui-inline-adder): component improvements
<oui-field label="Field 2" size="m">
<label class="oui-select">
<select name="field2" ng-model="field2" class="oui-select__input" required>
<option value="" disabled selected>Select the OS</option>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<option value="" disabled selected>Select the OS</option>
<option value="" disabled selected>Select the Operating System</option>

<select name="field2" ng-model="field2" class="oui-select__input" required>
<option value="" disabled selected>Select the OS</option>
<option value="freebsd">FreeBSD</option>
<option value="linux">Linux</option>
Copy link
Contributor

Choose a reason for hiding this comment

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

As you may know, Linux is not an operating system. You probably mean a GNU/Linux distributions right? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll update the example. It comes from the past contribution.

@AxelPeter AxelPeter merged commit 9824bbb into develop Nov 5, 2018
@AxelPeter AxelPeter deleted the refactor/oui-inline-adder branch November 5, 2018 13:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants