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/2709 v5.5.0 #2711

Merged
merged 65 commits into from
May 12, 2020
Merged
Show file tree
Hide file tree
Changes from 52 commits
Commits
Show all changes
65 commits
Select commit Hold shift + click to select a range
87a4534
issue/2709 Converted some classes to ES6 for future works
oliverfoster Mar 31, 2020
80787b1
issue/2709 fixed locking model for ES6 classes
oliverfoster Mar 31, 2020
5a686db
Recommendations
oliverfoster Apr 1, 2020
603d332
Recommendations
oliverfoster Apr 1, 2020
c368971
Recommendations
oliverfoster Apr 1, 2020
ce81506
Recommendation
oliverfoster Apr 1, 2020
6681597
Recommendation
oliverfoster Apr 1, 2020
d50d146
Recommendations
oliverfoster Apr 1, 2020
d4aad08
issue/2712 Simplfied router
oliverfoster Apr 1, 2020
8933d4e
Fixed bugs added comments
oliverfoster Apr 1, 2020
0375144
Added defer to postRemove events as expected
oliverfoster Apr 1, 2020
4db8f12
Removed bad condition
oliverfoster Apr 1, 2020
068d28d
Switched to cached value
oliverfoster Apr 1, 2020
551d458
Recommendations
oliverfoster Apr 2, 2020
4911dbb
Recommendations
oliverfoster Apr 2, 2020
749353f
Recommendations
oliverfoster Apr 2, 2020
81cacf1
issue/2714 Decoupled menu, page, article, block and component
oliverfoster Apr 2, 2020
64274b7
Typo
oliverfoster Apr 2, 2020
2de000f
Recommendations
oliverfoster Apr 4, 2020
1942b1a
issue/2714: Added Adapt.log.warnOnce
oliverfoster Apr 6, 2020
347f520
Linting fixes
oliverfoster Apr 6, 2020
96bda87
issue/2714 Added fallback model/view name discovery
oliverfoster Apr 6, 2020
ebdabc5
issue/2645 Added manifest creation and loading
oliverfoster Apr 6, 2020
dfb9224
Fixed issue with new compatibility layer changes
oliverfoster Apr 7, 2020
3b929b3
issue/2712 Simplfied router
oliverfoster Apr 1, 2020
df93b45
Fixed bugs added comments
oliverfoster Apr 1, 2020
763ccfc
Added defer to postRemove events as expected
oliverfoster Apr 1, 2020
eb6597a
Removed bad condition
oliverfoster Apr 1, 2020
eca527b
Switched to cached value
oliverfoster Apr 1, 2020
5a1cae8
Recommendations
oliverfoster Apr 2, 2020
c5b5bc4
Recommendations
oliverfoster Apr 2, 2020
a555e94
Recommendations
oliverfoster Apr 2, 2020
9fabc27
Merge branch 'issue/2712' of https://github.com/adaptlearning/adapt_f…
oliverfoster Apr 7, 2020
9a20e9b
issue/2714 Decoupled menu, page, article, block and component
oliverfoster Apr 2, 2020
53264e5
Typo
oliverfoster Apr 2, 2020
76cdf72
Recommendations
oliverfoster Apr 4, 2020
818dd05
issue/2714: Added Adapt.log.warnOnce
oliverfoster Apr 6, 2020
2bb469b
Linting fixes
oliverfoster Apr 6, 2020
b4abd4b
issue/2714 Added fallback model/view name discovery
oliverfoster Apr 6, 2020
be03dec
Merge branch 'issue/2714' of https://github.com/adaptlearning/adapt_f…
oliverfoster Apr 7, 2020
a861913
issue/2645 Added manifest creation and loading
oliverfoster Apr 6, 2020
c1130ae
Merge branch 'issue/2645' of https://github.com/adaptlearning/adapt_f…
oliverfoster Apr 7, 2020
d6dfe8d
Recommendations
olivermartinfoster Apr 14, 2020
4dd0ba8
Improved babel source mapping
olivermartinfoster Apr 14, 2020
3c7b656
Readded sourceType to babel config
olivermartinfoster Apr 14, 2020
cb1ee71
issue/2645 Added manifest creation and loading
olivermartinfoster Apr 14, 2020
cf924ca
isTypeGroup warning simplified
olivermartinfoster Apr 14, 2020
61c1a97
issue/2714 Decoupled menu, page, article, block and component
olivermartinfoster Apr 20, 2020
99d3aae
issue/2712 Simplified router
olivermartinfoster Apr 20, 2020
74e2b6c
Fix for authoring tool assets.json
olivermartinfoster Apr 21, 2020
9776793
Fixed linting issues
olivermartinfoster Apr 21, 2020
2af5448
Reworked AAT assets.json fix
olivermartinfoster Apr 21, 2020
e5e5157
Switched to native filter
olivermartinfoster Apr 21, 2020
014c62a
Switched from instanceof Array to Array.isArray
olivermartinfoster Apr 21, 2020
9529833
Fixed babel watch command
olivermartinfoster Apr 21, 2020
4d9a5c8
Added contentObject postRemove instance event, added global preReady …
olivermartinfoster Apr 21, 2020
3b0683c
Switched buttons and heading views to use contentObject postRemove in…
olivermartinfoster Apr 21, 2020
e756447
Added ready guard and removed duplicate preReady event
olivermartinfoster Apr 22, 2020
f1e632d
Moved _isReady reset back into Adapt.remove to allow multiple page re…
olivermartinfoster Apr 27, 2020
1c8ca18
ItemsQuestionModel not restoring question answers correctly, missing …
olivermartinfoster May 1, 2020
d7c85c4
resolved merge conflict
olivermartinfoster May 6, 2020
5cf0fa2
prevent invalid value from being processed by setActiveItem
moloko May 7, 2020
9f39b31
amend following code review
moloko May 7, 2020
bfea0cb
restored parameter
moloko May 7, 2020
29c0630
Merge pull request #2751 from adaptlearning/issue/2749
tomgreenfield May 11, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
73 changes: 51 additions & 22 deletions grunt/config/babel.js
Original file line number Diff line number Diff line change
@@ -1,30 +1,59 @@
module.exports = {
options: {
sourceMap: true,
inputSourceMap: true,
sourceType: 'script',
minified: true,
comments: false,
presets: [
[
'@babel/preset-env',
{
"targets": {
"ie": "11"
compile: {
options: {
inputSourceMap: false,
sourceType: 'script',
minified: true,
comments: false,
presets: [
[
'@babel/preset-env',
{
targets: {
ie: '11'
}
}
}
]
]
]
},
files: [{
expand: true,
cwd: '<%= tempdir %>',
src: [
'adapt.min.js'
],
dest: '<%= outputdir %>adapt/js/',
ext: '.min.js'
}]
},
dist: {
dev: {
options: {
sourceMap: true,
inputSourceMap: true,
sourceType: 'script',
retainLines: true,
minified: false,
compact: false,
comments: true,
presets: [
[
'@babel/preset-env',
{
targets: {
ie: '11'
}
}
]
]
},
files: [{
"expand": true,
"cwd": "<%= tempdir %>",
"src": [
"adapt.min.js"
expand: true,
cwd: '<%= tempdir %>',
src: [
'adapt.min.js'
],
"dest": "<%= outputdir %>adapt/js/",
"ext": ".min.js"
dest: '<%= outputdir %>adapt/js/',
ext: '.min.js'
}]
}
};
};
4 changes: 2 additions & 2 deletions grunt/config/watch.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ module.exports = {
tasks: ['handlebars', 'javascript:dev']
},
courseJson: {
files: ['<%= sourcedir %>course/**/*.<%= jsonext %>'],
tasks: ['jsonlint', 'check-json', 'newer:copy:courseJson', 'schema-defaults']
files: ['<%= sourcedir %>course/**/*.<%= jsonext %>', '<%= outputdir %>course/*/language_data_manifest.js'],
tasks: ['language-data-manifests', 'jsonlint', 'check-json', 'newer:copy:courseJson', 'schema-defaults']
},
courseAssets: {
files: ['<%= sourcedir %>course/<%=languages%>/*', '!<%= sourcedir %>course/<%=languages%>/*.<%= jsonext %>'],
Expand Down
12 changes: 8 additions & 4 deletions grunt/helpers/data/Language.js
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,10 @@ class Language {
const relativePath = dataFilePath.slice(this.path.length);
return relativePath;
}).filter((dataFilePath) => {
const isNotManifest = (dataFilePath !== this.manifestPath);
return isNotManifest;
const isManifest = (dataFilePath === this.manifestPath);
// Skip file if it is the Authoring Tool import/export asset manifest
const isAATAssetJSON = (dataFilePath === 'assets.json');
return !isManifest && !isAATAssetJSON;
});

dataFiles.forEach(jsonFileName => {
Expand Down Expand Up @@ -121,8 +123,10 @@ class Language {
const relativePath = dataFilePath.slice(this.path.length);
return relativePath;
}).filter((dataFilePath) => {
const isNotManifest = (dataFilePath !== this.manifestPath);
return isNotManifest;
const isManifest = (dataFilePath === this.manifestPath);
// Skip file if it is the Authoring Tool import/export asset manifest
const isAATAssetJSON = (dataFilePath === 'assets.json');
return !isManifest && !isAATAssetJSON;
});
const hasNoDataFiles = !dataFiles.length;
if (hasNoDataFiles) {
Expand Down
3 changes: 2 additions & 1 deletion grunt/tasks/build.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,11 @@ module.exports = function(grunt) {
'build-config',
'copy',
'schema-defaults',
'language-data-manifests',
'handlebars',
'tracking-insert',
'javascript:compile',
'babel',
'babel:compile',
'clean:dist',
'less:compile',
'replace',
Expand Down
3 changes: 2 additions & 1 deletion grunt/tasks/dev.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,11 @@ module.exports = function(grunt) {
'build-config',
'copy',
'schema-defaults',
'language-data-manifests',
'handlebars',
'tracking-insert',
'javascript:dev',
'babel',
'babel:dev',
'less:dev',
'replace',
'scripts:adaptpostbuild',
Expand Down
3 changes: 2 additions & 1 deletion grunt/tasks/diff.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,11 @@ module.exports = function(grunt) {
'build-config',
'copy',
'schema-defaults',
'language-data-manifests',
'newer:handlebars:compile',
'tracking-insert',
'newer:javascript:dev',
'babel',
'babel:dev',
'newer:less:dev',
'replace',
'scripts:adaptpostbuild',
Expand Down
7 changes: 7 additions & 0 deletions grunt/tasks/language-data-manifests.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
module.exports = function(grunt) {
const Helpers = require('../helpers')(grunt);
grunt.registerTask('language-data-manifests', 'Creates a manifest for each set of language data files', function() {
const languages = Helpers.getFramework({ useOutputData: true }).getData().languages;
languages.forEach(language => language.saveManifest());
});
};
3 changes: 2 additions & 1 deletion grunt/tasks/server-build.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,11 @@ module.exports = function(grunt) {
'_log-vars',
'build-config',
'copy',
'language-data-manifests',
'less:' + requireMode,
'handlebars',
'javascript:' + requireMode,
'babel',
'babel:' + requireMode,
'replace',
'scripts:adaptpostbuild',
'clean:temp'
Expand Down
6 changes: 3 additions & 3 deletions src/core/js/a11y.js
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ define([
Adapt.on('device:changed', this._setupNoSelect);
this.listenTo(Adapt, {
'router:location': this._onNavigationStart,
'pageView:ready menuView:ready router:plugin': this._onNavigationEnd
'contentObjectView:ready router:plugin': this._onNavigationEnd
});
},

Expand Down Expand Up @@ -137,7 +137,7 @@ define([
}
// Stop document reading
_.defer(function() {
Adapt.a11y.toggleHidden('.page, .menu', true);
Adapt.a11y.toggleHidden('.contentobject', true);
});
},

Expand All @@ -147,7 +147,7 @@ define([
return;
}
// Allow document to be read
Adapt.a11y.toggleHidden('.page, .menu', false);
Adapt.a11y.toggleHidden('.contentobject', false);
},

isActive: function() {
Expand Down
Loading