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

Jooa11y/enhancements #132

Merged
merged 2 commits into from
Nov 29, 2021

Conversation

brianteeman
Copy link
Owner

Pull Request for Issue # .

Summary of Changes

Testing Instructions

Expected result

Actual result

Documentation Changes Required

// Instantiate
const checker = new Jooa11y.Jooa11y(Jooa11yLangEn.options);
checker.doInitialCheck();
});
Copy link

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();
Copy link

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);
Copy link

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
Copy link

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);
Copy link

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
Copy link

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;
}
Copy link

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;
Copy link

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) {
Copy link

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
Copy link

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

@brianteeman brianteeman merged commit caff6bf into brianteeman:4.1-dev Nov 29, 2021
brianteeman added a commit that referenced this pull request Jan 1, 2022
* 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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants