Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
e2980042
Commit
e2980042
authored
Aug 02, 2017
by
Ahsan Ulhaq
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Accessibility issues on LMS Dashboard Social links
LEARNER-2105
parent
cd8d31b7
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
6 additions
and
10 deletions
+6
-10
common/test/acceptance/tests/lms/test_lms_dashboard.py
+0
-4
lms/templates/dashboard/_dashboard_course_listing.html
+6
-6
No files found.
common/test/acceptance/tests/lms/test_lms_dashboard.py
View file @
e2980042
...
...
@@ -158,8 +158,6 @@ class LmsDashboardPageTest(BaseLmsDashboardTest):
"
%3
Futm_campaign
%3
Dsocial-sharing
%26
utm_medium
%3
Dsocial-post
%26
utm_source
%3
Dtwitter"
)
self
.
assertEqual
(
twitter_widget
.
attrs
(
'title'
)[
0
],
'Share on Twitter'
)
self
.
assertEqual
(
twitter_widget
.
attrs
(
'data-tooltip'
)[
0
],
'Share on Twitter'
)
self
.
assertEqual
(
twitter_widget
.
attrs
(
'aria-haspopup'
)[
0
],
'true'
)
self
.
assertEqual
(
twitter_widget
.
attrs
(
'aria-expanded'
)[
0
],
'false'
)
self
.
assertEqual
(
twitter_widget
.
attrs
(
'target'
)[
0
],
'_blank'
)
self
.
assertIn
(
twitter_url
,
twitter_widget
.
attrs
(
'href'
)[
0
])
self
.
assertIn
(
twitter_url
,
twitter_widget
.
attrs
(
'onclick'
)[
0
])
...
...
@@ -170,8 +168,6 @@ class LmsDashboardPageTest(BaseLmsDashboardTest):
"quote=I
%27
m+taking+Test"
)
self
.
assertEqual
(
facebook_widget
.
attrs
(
'title'
)[
0
],
'Share on Facebook'
)
self
.
assertEqual
(
facebook_widget
.
attrs
(
'data-tooltip'
)[
0
],
'Share on Facebook'
)
self
.
assertEqual
(
facebook_widget
.
attrs
(
'aria-haspopup'
)[
0
],
'true'
)
self
.
assertEqual
(
facebook_widget
.
attrs
(
'aria-expanded'
)[
0
],
'false'
)
self
.
assertEqual
(
facebook_widget
.
attrs
(
'target'
)[
0
],
'_blank'
)
self
.
assertIn
(
facebook_url
,
facebook_widget
.
attrs
(
'href'
)[
0
])
self
.
assertIn
(
facebook_url
,
facebook_widget
.
attrs
(
'onclick'
)[
0
])
...
...
lms/templates/dashboard/_dashboard_course_listing.html
View file @
e2980042
...
...
@@ -169,18 +169,18 @@ from util.course import get_link_for_about_page, get_encoded_course_sharing_utm_
share_text =
_("I'm
taking
{
course_name
}
online
with
edX
.
org
.
Check
it
out
!").
format
(
course_name=
course_overview.display_name_with_default)
query_params =
urllib.urlencode((('u',
facebook_share_url
),
('
quote
',
share_text
.
encode
('
utf-8
')),))
facebook_url =
'https://www.facebook.com/sharer/sharer.php?{query}'
.
format
(
query=
query_params)
share_msg =
_("Share
{
course_name
}
on
Facebook
").
format
(
course_name=
course_overview.display_name_with_default)
%
>
<a
data-tooltip=
"${_('Share on Facebook')}"
data-trigger=
"focus hover"
class=
"action action-facebook"
aria-haspopup=
"true"
aria-expanded=
"false"
href=
"${facebook_url}"
target=
"_blank"
title=
"${_('Share on Facebook')}"
data-course-id=
"${course_overview.id}"
onclick=
"window.open('${facebook_url}', '${share_window_name}', '${share_window_config}'); return false;"
>
<span
class=
"sr"
>
${
_('Facebook')
}
</span>
<span
class=
"sr"
>
${
share_msg
}
</span>
<span
class=
"fa fa-facebook"
aria-hidden=
"true"
></span>
</a>
% endif
...
...
@@ -190,18 +190,18 @@ from util.course import get_link_for_about_page, get_encoded_course_sharing_utm_
default_share_text =
_("I'm
taking
{
course_name
}
online
with
@
edxonline
.
Check
it
out
!").
format
(
course_name=
course_overview.display_name_with_default)
share_text =
urllib.quote_plus(share_settings.get('DASHBOARD_TWITTER_TEXT',
default_share_text
.
encode
('
utf-8
')))
twitter_url =
'https://twitter.com/intent/tweet?text='
+
share_text
+
'%
20
'
+
urllib
.
quote_plus
(
twitter_share_url
)
share_msg =
_("Share
{
course_name
}
on
Twitter
").
format
(
course_name=
course_overview.display_name_with_default)
%
>
<a
data-tooltip=
"${_('Share on Twitter')}"
data-trigger=
"focus hover"
class=
"action action-twitter"
aria-haspopup=
"true"
aria-expanded=
"false"
href=
"${twitter_url}"
target=
"_blank"
title=
"${_('Share on Twitter')}"
data-course-id=
"${course_overview.id}"
onclick=
"window.open('${twitter_url}', '${share_window_name}', '${share_window_config}'); return false;"
>
<span
class=
"sr"
>
${
_('Twitter')
}
</span>
<span
class=
"sr"
>
${
share_msg
}
</span>
<span
class=
"fa fa-twitter"
aria-hidden=
"true"
></span>
</a>
% endif
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment