Skip to content

update for h5bp#30

Open
sams wants to merge 25 commits into
ProLoser:masterfrom
sams:master
Open

update for h5bp#30
sams wants to merge 25 commits into
ProLoser:masterfrom
sams:master

Conversation

@sams
Copy link
Copy Markdown
Collaborator

@sams sams commented Oct 8, 2012

consistent with h5bp v4

  • update html conditional comments
  • update styles (spliting into main.css & normalize.css)
  • update scripts (rename libs to vendor, rename script.js to main.js)
  • centralize HtmlHelper css & script calls in layout
  • update jquery & modernizr
  • remove AutoView & ThemedAuto View

Sam Sherlock added 21 commits October 8, 2012 11:05
…ss and Simkimsia ClearCache; reformat array
… update ie condition html tag output to have classes inline with h5bp v4 updates
…of html helper (and commented AssetCompress versions too);

minor rework consolidate these calls to central location;
add version and fallback to js lib call
using js/vendor (in place of js/libs)
@ProLoser
Copy link
Copy Markdown
Owner

ProLoser commented Oct 9, 2012

Does the original H5BP project still have modernizr enabled by default? I don't actually think it's used that much and am starting to think we should leave it commented out?

@sams
Copy link
Copy Markdown
Collaborator Author

sams commented Oct 9, 2012

yes h5bp does have include modernizr.
commenting it out is no biggie, I do think it is extremely useful

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants