Skip to content

Commit

Permalink
correct duplicate property name that was preventing the 'Source URL' …
Browse files Browse the repository at this point in the history
…field from appearing in the AT (fixes #5)

also add missing underscore to the property name of the Aspect Ratio field, make the ariaRegion text a bit more specific and remove the _allowFullScreen option from the example.json since full screen mode is now always enabled in the HTML5 YouTube player
  • Loading branch information
moloko committed Jan 4, 2016
1 parent 19e7d16 commit d7cbee4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
1 change: 0 additions & 1 deletion example.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
"_modestBranding": true,
"_playbackQuality": "hd720",
"_showAnnotations": false,
"_allowFullscreen": true,
"_progressColor": "red"
},
"_pageLevelProgress": {
Expand Down
6 changes: 3 additions & 3 deletions properties.schema
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"ariaRegion": {
"type": "string",
"required": true,
"default": "This is a media component which plays a video. Select the play / pause button to watch or listen.",
"default": "This is a media component which displays a YouTube video. Select the play / pause button to watch it.",
"inputType": "Text",
"validators": []
},
Expand Down Expand Up @@ -50,7 +50,7 @@
"required":true,
"title": "Media settings",
"properties":{
"source": {
"_source": {
"type":"string",
"required":true,
"default": "//www.youtube.com/embed/6qzOMCIlzt4",
Expand All @@ -59,7 +59,7 @@
"validators": [],
"help": "The 'embed' URL of the YouTube video you want to be displayed"
},
"source": {
"_aspectRatio": {
"type":"number",
"required":false,
"default": "1.778",
Expand Down

0 comments on commit d7cbee4

Please sign in to comment.