Closed moollaza closed 9 years ago
I’ll see what I can do…
Seems reasonable. Can you approve the suggestion text below for the rainfall IA.
Title: 534mm Subtitle: Australia - Average annual rainfall in millimeters
Expatisan suggestion
Current Title: Sydney cost of living index = 222 Subtitle: in Expatistan's scale (100 = Prague, Czech Republic)
Suggestion Title: 222 Subtitle: Sydney - Cost of living - Expatistan's scale (100 = Prague, Czech Republic)
The URL shorten IA uses a custom template currently to display the editable input box. Guessing you'd just like the "Shortened URL" appended below the input box? Please clarify.
@MrChrisW these look pretty good to me, I see you made a PR so I'll comment over there. Thanks!
@moollaza @MrChrisW Anything else to do here?
@javathunderman I don't think so, it looks like @MrChrisW managed to get to everything already!
@javathunderman hehe if you're looking for work I made a few new issues today about reviving some old Spice IAs. I'd love to see Forvo up and running again.
Right, looking into that!
The URL shorten IA uses a custom template currently to display the editable input box. Guessing you'd just like the "Shortened URL" appended below the input box? Please clarify.
@MrChrisW yup that sounds perfect!
@moollaza Ok, I've had a play around with the shorten link design. I don't think we really need to display "Shortened URL" under the link as it's on the answer bar, it feels repetitive. Maybe if the answer bar, was just "Answer" then we could provide a subtitle. Anyway see some examples below.
Current selection_201
Suggested selection_202
Improved selection_200 Improved + Suggested selection_199
@MrChrisW looking good! Maybe we should switch URL with the actual URL that we shortened? Then it's less redundant.
@abeyang any thoughts on how the "URL Shorten" IA should look?
@MrChrisW let's go with the standard title / subtitle, but with the input field included:
Couple of things to note:
LMK if you have any thoughts / questions!
@abeyang Updated as per your recommendations :+1: - View online
Sweet @MrChrisW -- almost there! Just need a few more css tweaks:
.zci--shorten input.tag {
font-size: 1.2em;
}
.zci__subheader, .zci__header__sub {
margin-top: 2px;
color: #999;
}
@abeyang Updated as per your request.
One question, why are we changing these default values? You can just say "for design purposes" and I'll shut up :laughing:
.zci__header {
font-size: 1.38em
}
.zci__subheader, .zci__header__sub {
color: #888;
}
Looks good! :+1:
That's a good question. We're still in the process of standardizing things, including going back and revisiting existing templates. Until we actually make those changes in the templates, it's going to require some hand-tweaking here and there...
But with that said, for the subheader, I think #888 would've been okay -- but yeah, looks good!
Thanks for the explanation @abeyang Looking forward to seeing styles standardised :smiley:
@moollaza This should be good to close, provide there are no other fact IA's? Will have to remember if any new fact Spices appear to stick to these guidelines! Thanks
@MrChrisW apologies for the delay, will tackle this ASAP.
Haha whoops, thought this was a PR last night. We can close, the IA's have all been updated :+1:
Will have to remember if any new fact Spices appear to stick to these guidelines! Thanks
@MrChrisW we're also updating our templates and adding some new ones. I expect these kinds of IA's will be covered under a new template :)
In an effort to further standardize our IA displays, we'd like to update IA's that return factual data like numbers, to display the number prominently at the top, with a short subtitle to provide context.
For example the Sales Tax IA should show:
6%
Pennsylvania - State Tax
The following IA's should be updated:
/cc @abeyang @jagtalon