duckduckgo / zeroclickinfo-spice

DuckDuckGo Instant Answers based on JavaScript (JSON) APIs
https://duckduckhack.com/
Other
548 stars 942 forks source link

Minor formatting changes to Wordnik IA #1074

Closed chrismorast closed 10 years ago

chrismorast commented 10 years ago

@jagtalon , @jdorweiler , @moollaza , @MariagraziaAlastra

I'd like to make a few formatting changes on the wording IA to improve the format.

  1. Increase bottom margin on content so that it visually matches the top margin (represented in pink on the screenshot). This may need to be reduced from what I have in the screenshot but we can explore that.
  2. Increase bottom padding on definitions to .75em so it visually matches the top (represented in blue on the screenshot).
  3. Indent the definition to separate it from the definition type (i.e. n., v, adv., adj...) so that it's more tabular. (spacing should be equal to about .75em or 1em) This would also apply to definitions that wrap 2-lines or more. The wrapped lines would match the indentation of the first line. This will help with parsing the word type more easily and clean up the UI where definition wrap several lines (represented in yellow on the screenshots).

Any chance one of you could tackle this?

intended look margins and padding wrapping line indentation

MariagraziaAlastra commented 10 years ago

@chrismorast It will be my pleasure to take care of this :)

chrismorast commented 10 years ago

thank you @MariagraziaAlastra , just saw the update... it looks great!

jagtalon commented 10 years ago

Fixed in https://github.com/duckduckgo/zeroclickinfo-spice/pull/1085