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

issue #985 #992

Closed
wants to merge 3 commits into from
Closed

issue #985 #992

wants to merge 3 commits into from

Conversation

Mariatta
Copy link
Member

Attempt to address issue #985

removed braile, embossed, tty value from media attribute since they've been deprecated
replaced inline table attributes: cellpadding, cellspacing, width, border, used css
adjust values for role attribute
These two errors remained, but I'm not familiar with these. Any suggestion of what to do here?
Bad value cleartype for attribute http-equiv on element meta.
Bad value imagetoolbar for attribute http-equiv on element meta.

fix issue with role attributes
@@ -13,15 +13,15 @@ <h2 class="widget-title"><span aria-hidden="true" class="icon-success-stories"><
<a href="{{ story.get_absolute_url }}">{{ story.pull_quote }}</a>
</blockquote>

<table cellpadding="0" cellspacing="0" border="0" width="100%" class="quote-from">
<table class="quote-from">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for cleaning up this!

@berkerpeksag
Copy link
Member

Looks good to me, thanks! I still need to test this in my local setup.

Bad value cleartype for attribute http-equiv on element meta.

This is probably harmless and can stay.

Bad value imagetoolbar for attribute http-equiv on element meta.

This is for IE's image toolbar and we can probably remove it. It would be nice to check if it's still useful in Edge.

@Mariatta
Copy link
Member Author

Mariatta commented Sep 8, 2016

Hi, just wanted to follow up on this pull request. Wondering if I should make additional adjustments?
Thanks.

@berkerpeksag
Copy link
Member

Sorry, I'm pretty busy with Python 3.6.0 b1 this week (deadline is on 12th) I will try to get this in next week, thanks! In the meantime, it would be great if you can remove the "imagetoolbar" tag I mentioned in #992 (comment).

@Mariatta
Copy link
Member Author

Mariatta commented Sep 8, 2016

No problem. I will get the imagetoolbar tag removed later today.
Thanks.

@Mariatta
Copy link
Member Author

Let me know anything else that I should improve on this.
Is there a process in place for testing against different browsers?
Thank you.

@berkerpeksag
Copy link
Member

@Mariatta do you have time to rebase the validate-html branch? (and apologies for my very late response!)

@Mariatta
Copy link
Member Author

Mariatta commented Jul 6, 2018

😅 I also forgot about this PR. If you're in no rush, I'll be able to get back to this in a couple more weeks.

@berkerpeksag
Copy link
Member

Of course! I can also work on it, I still have a few hours left from my grant :)

@Mariatta
Copy link
Member Author

Mariatta commented Jul 6, 2018

@berkerpeksag Sounds awesome! I really don't mind if you can take this over.
I'm on a new laptop and will take more time for me to get my local environment setup again.

@JacobCoffee
Copy link
Member

Closing in favor of #2536 since I can't push to forks.

Commit credits will remain.

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.

3 participants