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

Better IDL list interface #21

Merged
merged 5 commits into from
Oct 22, 2015
Merged

Better IDL list interface #21

merged 5 commits into from
Oct 22, 2015

Conversation

andrewdeandrade
Copy link
Contributor

Solves #14, #16, #17 and #18

@breerly @kriskowal @Raynos

@HelloGrayson
Copy link

@malandrew yo dude! try and submit changes in separate, whole, and cohesive PR's in the future!

@HelloGrayson
Copy link

Can you attach a screenshot of the new idl list command?

@andrewdeandrade
Copy link
Contributor Author

screen shot 2015-10-21 at 2 50 13 pm

@HelloGrayson
Copy link

Looking good. Let's:

  • remove the underline on the column headers, the uppercase and blue is enough.
  • you don't need hyphens for empty values in the LOCAL column, causes the ui to be busy.

andrewdeandrade pushed a commit that referenced this pull request Oct 22, 2015
@andrewdeandrade andrewdeandrade merged commit e9461f4 into master Oct 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants