-
-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
css for SQL lessons #9
Comments
Hi, @orchid00. I can't find challenges in the repository with lesson on SQL. Can you point me at them? |
http://github.com/swcarpentry/sql-novice-survey has a bunch of challenges.
|
@gvwilson do you both mean "FIXME"s? |
I meant the font of the titles called "challenges" so all of them. It is a matter of style: they are very spacious in between, and probably the question title (where it says fixme) should be a bit smaller. Is that clearer? |
Thanks @orchid00 and @gvwilson. My fork, for reason unknown to me, contained "FIXME" headers instead of proper challenges as, for example, here http://swcarpentry.github.io/sql-novice-survey/01-select.html (at the bottom). I've pulled in the recent changes and will work on styles some more. |
Hi everyone, I don't like that light-green color of challenge boxes. I'd get rid of it, and to highlight that this section is a challenge I'd put it under "Challenge" (or "Exercise") header. Pros:
Cons:
What do you think? |
Oh! tables look much better like that. Also, we talked with some other instructors that the challenges should come more often and not only at the end of the lesson. But I guess that is a more structural change. In that case wouldn't be necessary to write challenge headers :P |
Oh I really like this change.
You're right. This gives me an idea of how to bring nice-looking exercises inside the lesson's content. Take a look at this "Note" section here: it looks like a box with a header, but it's not disturbing. I'm thinking a similar style for challenges: it has to be a box, because challenges sometimes get more than one paragraph of text, and it must not be disturbing, ie. header should be fairly small and colors not too bright. I'll probably throw away my previous changes to challenges and prepare something that presents these exercises nicely and inline, not at the end of the lesson. |
Yeah the box with the header looks OK. I think before it was just the color green that didn't look so nice. Again, I see something different, your queries are now in grey-ish? and my queries are color green... |
The queries in grey-ish had no |
@pbanaszkiewicz (I'm new here) where should I do the PR? |
@orchid00 that's alright. I think the issue is that code blocks in challenges miss
Here's an issue ticket I've just created: swcarpentry/sql-novice-survey#34 Please send a PR to the https://github.com/swcarpentry/sql-novice-survey/ repository. |
[RDY] pandoc filter to convert blockquotes to divs
01-tooling: Fix 'jeyll-collection' -> 'jekyll-collection' typo
Edit text since there is no longer any red formatting in table.
Hi,
I have some suggestions on the template for the SQL lessons:
Padding:
*I see that most if not all tables need a bit more of bottom padding, the letters in the next paragraph are too close.
*Challenges on the other side are very spacious in between, and probably the question title should be a bit smaller.
Colors:
*Table headers, I don't know if it is just me, but that blue is too dark to be combined with black... just saying...
This is a duplicate from:
swcarpentry/sql-novice-survey#27
suggested by Raniere Silva to be copied here.
The text was updated successfully, but these errors were encountered: