attend page cleanup - Githubissues

code4lib / 2017.code4lib.org

Code4lib 2017 Los Angeles

http://2017.code4lib.org

4 stars 13 forks source link

attend page cleanup #104

Closed phette23 closed 7 years ago

phette23 commented 7 years ago

Just a few things I've noticed, a lot of people are about to hit this page:

[x] PR #103 fixes a missing period & the election URL results
[x] "Only the top ten (highlighted) have been accepted so far" how are the proposals highlighted? I didn't notice any indicator
[ ] would be nice to colorize the lanyard colors (red, yellow, green)
[ ] can we link up something in the Hotel Fees section such as the housing link that's in the registration form?

queryluke commented 7 years ago

RE: second bullet, should it link to the accepted talks? Is this list valid? http://2017.code4lib.org/talks/

phette23 commented 7 years ago

Hmm that's a good question. Sentence says top 10 but that's more than ten listed. A question for people in Slack maybe? It seemed deliberate that the link was to the election results.

queryluke commented 7 years ago

Ah, context is important. Good point. I'm guessing just need to remove the "highlighted part"

gmcharlt commented 7 years ago

That it links to the election results is just an accident of timing, as /schedule/ was not ready when I initially updated /attend/. I think that item can now be adjusted to say something along the lines of "The schedule is available" and link to /schedule/.

highermath commented 7 years ago

http://2017.code4lib.org/talks/ looks correct. We should just kill the highlighted part. I would do it, but jekyll died on my workstation.

queryluke commented 7 years ago

I'm on it

gmcharlt commented 7 years ago

Yeah, /talks/ is more concise for the purpose than /schedule/.

phette23 commented 7 years ago

I think the attend page looks fine these days, closing this just so there's less noise in the issues.