-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Jooa11y/enhancements #132
Jooa11y/enhancements #132
Conversation
// Instantiate | ||
const checker = new Jooa11y.Jooa11y(Jooa11yLangEn.options); | ||
checker.doInitialCheck(); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Newline required at end of file but not found eol-last
|
||
// Instantiate | ||
const checker = new Jooa11y.Jooa11y(Jooa11yLangEn.options); | ||
checker.doInitialCheck(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected indentation of 2 spaces but found 4 indent
Jooa11y.Lang.addI18n(Jooa11yLangEn.strings); | ||
|
||
// Instantiate | ||
const checker = new Jooa11y.Jooa11y(Jooa11yLangEn.options); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Jooa11y' is not defined no-undef
'Jooa11yLangEn' is not defined no-undef
Expected indentation of 2 spaces but found 4 indent
// Set translations | ||
Jooa11y.Lang.addI18n(Jooa11yLangEn.strings); | ||
|
||
// Instantiate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected indentation of 2 spaces but found 4 indent
} | ||
|
||
// Set translations | ||
Jooa11y.Lang.addI18n(Jooa11yLangEn.strings); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Jooa11y' is not defined no-undef
'Jooa11yLangEn' is not defined no-undef
Expected indentation of 2 spaces but found 4 indent
return; | ||
} | ||
|
||
// Set translations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected indentation of 2 spaces but found 4 indent
// Check if Jooa11y is loaded | ||
if (!Jooa11y) { | ||
return; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected indentation of 2 spaces but found 1 tab indent
Unexpected tab character no-tabs
window.addEventListener('load', () => { | ||
// Check if Jooa11y is loaded | ||
if (!Jooa11y) { | ||
return; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected indentation of 4 spaces but found 2 tabs indent
Unexpected tab character no-tabs
@@ -0,0 +1,13 @@ | |||
window.addEventListener('load', () => { | |||
// Check if Jooa11y is loaded | |||
if (!Jooa11y) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Jooa11y' is not defined no-undef
Expected indentation of 2 spaces but found 1 tab indent
Unexpected tab character no-tabs
@@ -0,0 +1,13 @@ | |||
window.addEventListener('load', () => { | |||
// Check if Jooa11y is loaded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected indentation of 2 spaces but found 1 tab indent
Unexpected tab character no-tabs
* jooa11y scaffolding media folder web assets hard coded until we decide on npm or not to install check out this branch npm run build:js npm run build:css discover install the plugin todo - aka brian needs help [] make the checker start [] make the checker use the settings from the plugin * script loads now * tippy - temp * editor button * toolbar * enpugh * Jooa11y/enhancements (#132) * Add some enhancements * Fix wrong name Co-authored-by: Benjamin Trenkle <[email protected]> * tabs * tippy popper licence and naming * postgresql install * xml bug * cleaning * tidy * fix * dont need scss * needs to be an integer * deploy version * might not be the best way? * Revert "Auxiliary commit to revert individual files from 18199dd" This reverts commit 4eb15800ccc4bb0d64580013b58a92505bc5abfc. * clarify show * options * package.lock * remove editor button * cs * string * set input * deploy * Temporary add git to node ci (#133) * github link * param * Revert "param" This reverts commit 378a64e. * params * spelling * revert * sql field * cs and spelling * Fix npm * Revert drone.yml change * Switch package to tarball * Use the es module * Update eslint-plugin-vue to 8.2.0 * CS (#135) * lang * progress update * fix strings * load strings * js change from @Fedik * typo and package-lock * sef and non-sef * revert change by Dimitris * Update joomla-a11y-checker version * Update joomla-a11y-checker version * Update view preview url * simplify * Update jooa11y.php * fix jscs (#136) * undo eslint change * remove "Additional ruleset toggles" from the plugin settings * Update .eslintrc lets try this again * final try * sha Co-authored-by: Benjamin Trenkle <[email protected]> Co-authored-by: Harald Leithner <[email protected]> Co-authored-by: dgrammatiko <[email protected]> Co-authored-by: Dimitris Grammatikogiannis <[email protected]> Co-authored-by: Benjamin Trenkle <[email protected]> Co-authored-by: Fedir Zinchuk <[email protected]>
Pull Request for Issue # .
Summary of Changes
Testing Instructions
Expected result
Actual result
Documentation Changes Required