-
Notifications
You must be signed in to change notification settings - Fork 18
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
Updating event-template.html #194
Comments
Just butting in on target="_blank": please don't - it's really user unfriendly. I know it's meant to help (I guess?) those who don't know they can open links in new tabs/windows, but what it really does is take any choice away from those who'd like to decide for themselves on how to open links. |
Apart from that, I think it's a good idea to update the template! 👍 (I threw out quite a few bits of the current one when I created the page for our Vienna event because I didn't find them very usable...) |
Fair enough, I wasn't completely sure about that one. I'll start this
|
Sounds like a good idea! It's also what @lindaliukas suggested back when I said I might rewrite it (which I then never came around to, ahem). |
I've done some progress (not recently) on the template: make sure you see the source code (we want cleaner markup) and in different screen resolutions: http://leonelgalan.github.io/railsgirls-event_template/ I'm going to keep working on it this week. |
I've been playing around with the event template. If you like, you can take a look at http://rg.patrickbaselier.nl/event-template.html to view the result. I've used middleman for this since I don't like html so much, but I love haml, Sass and compass. I only worked on the event template, since this both has impact on the markup (although I tried to keep it as close to the original as possible) but also on the structure of the application and the deployment process. It needs some more work to restructure the home- and other pages. Hope you like it and if you consider using this, just let me know if I can be of any help. |
I like what you did here, except for HAML and Compass. I don't like HTML too (I'm using SLIM more often than HAML), but it introduces has a learning curve to steep for some. Compass, shouldn't be required. Minor things I noticed:
Maybe we can chat, and make the best of both solutions a single, stronger solution that can be reconsidered by Rails Girls. |
Are there any good/easy to understand tutorials for HAML? (Funnily enough, yesterday was the first time I opened a HAML document, and now I'm actually stuck editing it...) Otherwise I'd also stick to HTML because it's what most people know! |
Both slim and haml have their better documentation in their sites: http://slim-lang.com/ & http://haml.info/. They are not very good, but it's there! While doing slim, I found this to be useful: http://html2slim.herokuapp.com/ |
@leonelgalan: I agree with your objectives. Although pre-compressing can save a lot of time and can bring structure, it does introduce a learning curve. This also counts for Sass (and Compass), and currently I haven't thought about how to tackle this. Maybe a template can be produced using pre-processing tools and users are presented with a processed version? I intentionally didn't touch the existing markup (or as little as possible), but I agree, this can be more semantic. Your suggestion about combining our approaches sounds nice. |
I think this might be related to #557 since we could use Jekyll to generate most stuff. |
rg-header.psd
which I believe is no longer necessary because the addition of web fontsWe should suggest using target="_blank" for external links(Probably not the best idea, we won't do this one)I just want to touch base, and see if this is something that we are interested in pursuing this and if there is more people interested in collaborating.
The text was updated successfully, but these errors were encountered: