Skip to content
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

Add browser support tables #149

Merged
merged 6 commits into from
Oct 3, 2014

Conversation

betravis
Copy link
Contributor

@betravis betravis commented Oct 2, 2014

Create a template mechanism for adding browser support tables, and add an initial table for the Shapes page. Part of the fixup work for bug #84.

@betravis
Copy link
Contributor Author

betravis commented Oct 2, 2014

Went with this style of comment

desaturate-depad

@rhauck do you have the bandwidth to look at this?

@rhauck
Copy link
Collaborator

rhauck commented Oct 3, 2014

The colors & layout look good to me. However, the browser icons aren't rendering on phones:

image

@rhauck
Copy link
Collaborator

rhauck commented Oct 3, 2014

Also, we might want to centralize the browser support data rather than have it in each page so i can be used elsewhere on this site (like maybe as way to rebuild the enable page?)

@betravis
Copy link
Contributor Author

betravis commented Oct 3, 2014

Made the suggested updates. Browser logos are now separate svg files. The support data for shapes is also now in _data/browser_support.yml.

@rhauck
Copy link
Collaborator

rhauck commented Oct 3, 2014

Looks better, but the iOS Apple is disproportionately tiny:

image

…ables

* upstream/master:
  Update README to reflect SASS build changes.
  remove unnecessary changes
  more feedback
  More fb from Bear
  Incorporate Bear && Rebecca fb
  Incorporate Bear && Rebecca fb
@betravis
Copy link
Contributor Author

betravis commented Oct 3, 2014

Because the images changed to be different files, they were no longer scaling the same, which caused each cell to be as wide as its text. I've changed the CSS to explicitly size the table cells given the number of columns generated.

@rhauck
Copy link
Collaborator

rhauck commented Oct 3, 2014

Looks good now on both phones now. Do you want to add this to the rest of the pages in this PR or merge this one and do the other pages separately?

@rhauck
Copy link
Collaborator

rhauck commented Oct 3, 2014

Merging this one and logged another issue for the others.

rhauck added a commit that referenced this pull request Oct 3, 2014
@rhauck rhauck merged commit e712135 into adobe-webplatform:master Oct 3, 2014
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.

None yet

2 participants