Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #17294 from edx/HarryRein/a11y-footer-upgrade-div

Browse Source 
Ensuring div and not button inside link tag for a11y.
This commit is contained in:
Harry Rein
2018-01-24 11:18:04 -05:00
committed by GitHub
parent d716d7409c 98b8eaf27e
commit 2935578b9a
2 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
lms/static/sass/features/_course-sock.scss
View File

@@ -101,6 +101,12 @@
color: theme-color("inverse");
background-image: none;
box-shadow: none;
cursor: pointer;
&:hover {
background-color: theme-color("inverse");
color: theme-color("success");
}
@media (max-width: 960px) {
& {

4
openedx/features/course_experience/templates/course_experience/course-sock-fragment.html
View File

@@ -59,9 +59,9 @@ from openedx.features.course_experience import DISPLAY_COURSE_SOCK_FLAG
% endif
<img class="mini-cert" alt="Example Certificate Image" src="${static.url('course_experience/images/verified-cert.png')}"/>
<a href="${upgrade_url}">
<button type="button" class="btn btn-upgrade stuck-top focusable action-upgrade-certificate" data-creative="original_sock" data-position="sock">
<div class="btn btn-upgrade stuck-top focusable action-upgrade-certificate" data-creative="original_sock" data-position="sock">
${Text(_('Upgrade ({course_price})')).format(course_price=HTML(course_price))}
</button>
</div>
</a>
</div>
</div>