Redesign: axe accessibility issues#11403
Merged
Conversation
ferblape
previously approved these changes
Jul 28, 2023
Contributor
ferblape
left a comment
There was a problem hiding this comment.
Code looks ok although there are some tests still failing
Contributor
Author
|
Solved |
ferblape
previously approved these changes
Jul 28, 2023
Contributor
|
@decidim/maintainers this PR solves a few general problems in common elements, which affected many many pages. Please take a look 🙏🏾 |
Contributor
|
@Crashillo Note that there are still some linter issues failing in the pipeline. There were also few other workflows failing, I re-run those as I'm not sure if they're flaky ones. |
ahukkanen
requested changes
Aug 8, 2023
Contributor
ahukkanen
left a comment
There was a problem hiding this comment.
Great work! I've checked through the pages and did not find any further issue apart from the one mentioned issue that will be fixed by #11426. The CI is also green now.
As the last round I've gone through the code and I've found one mistake and few improvement ideas. Could you still have a look at these? Then we can call it done!
decidim-forms/app/views/decidim/forms/questionnaires/answers/_matrix_multiple.html.erb
Show resolved
Hide resolved
decidim-forms/app/views/decidim/forms/questionnaires/answers/_matrix_single.html.erb
Show resolved
Hide resolved
ahukkanen
requested changes
Aug 9, 2023
Contributor
ahukkanen
left a comment
There was a problem hiding this comment.
There's still a couple of minor issues with the progress bars.
Plus we need to add proper translations to those.
decidim-budgets/app/views/decidim/budgets/projects/_order_progress_summary_content.html.erb
Outdated
Show resolved
Hide resolved
alecslupu
added a commit
to i-need-another-coffee/decidim
that referenced
this pull request
Aug 10, 2023
* Redesign: rename tests (decidim#11184) * Remove skip_unless_redesing_enabled calls in tests * Remove redesign_helpers for tests * Unskip and adapt REDESIGN_PENDING examples * Remove deprecated examples * Allow to configure the default header level of attributes titles in diff cell * Ensure accessibility of h in collaborative draft show * Recover accessibility tests in proposals * Remove comments * Unskip and adapt test * Fix expected text of link in test * Remove check for removed follow actions in participatory text index * Unskip tests * Unskip and adapt to vote proposal tests * Remove deprecated methods * Remove deprecations in tests * Remove collaborative drafts and proposals deprecated m cells * Adapt proposal m cell cache hash to l cell * Adapt proposal m-cell specs to l-cell * Replace use of proposal_m card * Remove condition on redesign from test * Rename shared examples for attachments to not use redesign term * Remove tests for CTA behavior deprecated in processes card-g * Avoid duplicated ids of highlighted participatory process in menu breadcrumb * Recover skipped examples * Recover accessibility tests in participatory processes * Remove participatory process description specs and include expected contents with metadata content block enabled * Unskip REDESIGN_PENDING tests * Move common statistics cells tests to shared examples * Remove unused process metadata item * Remove deprecated proceses m card and add tests for l and g cards * Remove deprecated process group m card cell * Recover tests after redesign in initiatives * Remove skipped deprecated tests * Remove assemblies description page test and move examples to show page with appropriate content blocks * Remove assembly m card and related tests * Unskip tests after completing redesing * Adapt sorting test to redesigned interface * Remove deprecated sorting tests because the redesigned interface prevents incomplete sortings * Unskip tests * Remove post m card cell * Remove reference to redesign from test * Remove comment m card * Unskip accessibility test in debates versions * Remove deprecated test * Remove deprecated comments about redesign * Remove deprecated test The proposals card-l does not show the supports count in the metadata, so there is no difference when supports are enabled on proposals component or not * Remove useless reference to redesign in test * Remove REDESIGN_PENDING comment: do not use card-l cell for media links * Remove REDESIGN_PENDING comments * Remove unused partial * Unskip tests * Remove conference m card cell * Adapt authorizations tests to redesign * Remove deprecated tests * Avoid errors in upload modal when file validation fails * Unskip tests * Remove deprecated test * Remove REDESIGN_PENDING comments * Update map styles in proposals * Uncomment tests * Uncomment and adapt selector in tests * Remove reference to redesign * Fix linter offense * Remove obsolete brakeman skip and update other in proposals * Remove unused translations and add some wizards keys to ignore unused * Update expected url in activity cell with comments activities * Increase header levels in version cell to include h1 * Recover missing data attribute in hero content block * Fix coordinates update in add_proposal js * Hide map only if there is no address with coordinates * Empty address field before changing it in test * Add issue reference to skipped tests * Display replies on deleted comments and keep then with AJAX deletion refresh * Unskip test * Update decidim-core/app/cells/decidim/version/show.erb Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Use h2 tag * Move shared examples require to core * Replace actions with method doing the same * Fix test * Remove references to redesign in shared examples --------- Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Redesign: refactor the scope of the main Javascript initializer (decidim#11317) * set the scope of all functions that are initialized at startup * scope the datepicker as well * remove precated js stuff (testing) * fix lint & place accordingly the initializations * Redesign: deprecate foundation-datepicker from public site (decidim#11420) * deprecate foundation-datepicker public site * fix core tests * fix datepicker tests * fix lint and rubocop * replace now with current * fix tests * fix more tests * fix tests * remove byebugs * Redesign: don't submit forms on modal's cancel * Update documentation for here map configuration (decidim#11447) * Two-columns layout fix (decidim#11400) * Column conversations fix * suggested changes added to &__aside * &__main updated * Changes made to span on &__main * test * spacing cli test * Disabler for pointer events (decidim#11449) * Added a disabler for pointer event * Improved workaround for the tabs +1 * Indentation line 19 * Fix test component (decidim#11436) * Fix label position on report button (decidim#11435) * Fix pipeline after decidim#11449 (decidim#11455) * Upgrade postgres from 11 to 14 (decidim#11453) * Fix initiatives ransack filters (decidim#11329) * Revert "Lock ChromeDriver to the latest working version (decidim#11391)" (decidim#11458) This reverts commit 2450714. * Force capital starting letter for PR titles (decidim#11457) * Force capital starting letter for PR titles * Force the starting character to be capitalized * Allow disabling password expiration (decidim#11135) * Allow disabling password expiration NIST does not recommend expiring passwords: https://pages.nist.gov/800-63-FAQ/#q-b05 * Add documentation about the new option * Fix linter * Fix linter * Fix autocomplete result list duplicates (decidim#11442) * Fix autocomplete user group duplication / Add system test * Fix test * Redesign: organization colors (decidim#11451) * refactor the organization colors * typo * remove specs about them_color input * fix specs * use old red for testing * set the default redesign colors to the organization factory (color-contrast) * replace the uses of legacy primary color * Redesign: axe accessibility issues (decidim#11403) * dropdown card does not require an id * add aria-label to progress bar * disable dropdown for desktop navigation * Refactor the use of filter_text_for to be in views instead of helpers * Allow passing an id to filter_text_for method * remove element_id (testing) * pass the id to the label * distinguish filters and order controls * distinguish all uses of dropdown-menus * rename control * don't use a list when you need divs * disable dropdown for desktop navigation * update code for emoji * remove ids from tests * fix core tests * fix election tests * fix tests * remove profile__user-role class * remove needless aria-attribute * remove duplicated ids * empty merge * Include alternative title for images when there's nothing but the title * disable dropdown for desktop * add alt text to default image * user profile must not be an h5 * duplicate_id and copy button SR text * fix odd layout for edit initiative * redesign committee requests form * fix a11y issue on budgets project list * mark layout item header al role=complementary since it's outside * do not expand elements if the page has very few contents * fix a11y issue on debates, sortitions and forms * fix rubocop * fix erblint * use the units as the aria-label * standarize arias for progress bars * initiatives style glitch --------- Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com> * Integrate Bullet in decidim-dev (decidim#11416) * Initial commit on bullet integration * remove uneeded condition * Disable failing actions * Disable failing workflows * Disable all the workflows * Refactor actions * Revert generator change * Redesign: comment button (decidim#11461) * render the comment button depending of the settings * enhance condition: initiatives is not a component * handle comments_blocked? in case of null * fix offenses * refactor duplicated views (decidim#11459) * Redesign: remove the item itself from nav menu (decidim#11452) * remove the item itself from nav menu * fix i18n * fix specs * fix specs * Refactor Decidim jobs to inherit from Decidim::ApplicationJob (decidim#11468) * Add Initiatives to antispam * Add initiatives specs --------- Co-authored-by: Eduardo Martínez <eduardomech@gmail.com> Co-authored-by: Hugoren Martinako <aumpfbahn@gmail.com> Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com> Co-authored-by: Tom <101816158+greenwoodt@users.noreply.github.com> Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi> Co-authored-by: Carlo Beltrame <carlo@beltra.me> Co-authored-by: JoonasAapro <110532525+JoonasAapro@users.noreply.github.com>
fblupi
added a commit
to Platoniq/decidim
that referenced
this pull request
Aug 25, 2023
* Redesign: counteract the hover effects of the dropdown (decidim#11253) * hack to counteract the hover effects of the dropdown * keep menu unreachable onload * due to a bug in Chrome, move the inline events to script tags more info: https://bugs.chromium.org/p/chromium/issues/detail?id=868224&q=ontransitionend&colspe[%E2%80%A6]Block%20Component%20Status%20Owner%20Summary%20OS%20Modified * fix spelling * extract script to separate file * restore removed element * fix spelling * trigger manually dropdown controller on hover parent * move onmouseenter only to link * handle dropdowns only via css animation * no longer required * fix styles * disable animations on page load * fix tests * fix syntax * Filter parameters details (decidim#11418) * Tiny typo fix (decidim#11428) * Redesign: rename tests (decidim#11184) * Remove skip_unless_redesing_enabled calls in tests * Remove redesign_helpers for tests * Unskip and adapt REDESIGN_PENDING examples * Remove deprecated examples * Allow to configure the default header level of attributes titles in diff cell * Ensure accessibility of h in collaborative draft show * Recover accessibility tests in proposals * Remove comments * Unskip and adapt test * Fix expected text of link in test * Remove check for removed follow actions in participatory text index * Unskip tests * Unskip and adapt to vote proposal tests * Remove deprecated methods * Remove deprecations in tests * Remove collaborative drafts and proposals deprecated m cells * Adapt proposal m cell cache hash to l cell * Adapt proposal m-cell specs to l-cell * Replace use of proposal_m card * Remove condition on redesign from test * Rename shared examples for attachments to not use redesign term * Remove tests for CTA behavior deprecated in processes card-g * Avoid duplicated ids of highlighted participatory process in menu breadcrumb * Recover skipped examples * Recover accessibility tests in participatory processes * Remove participatory process description specs and include expected contents with metadata content block enabled * Unskip REDESIGN_PENDING tests * Move common statistics cells tests to shared examples * Remove unused process metadata item * Remove deprecated proceses m card and add tests for l and g cards * Remove deprecated process group m card cell * Recover tests after redesign in initiatives * Remove skipped deprecated tests * Remove assemblies description page test and move examples to show page with appropriate content blocks * Remove assembly m card and related tests * Unskip tests after completing redesing * Adapt sorting test to redesigned interface * Remove deprecated sorting tests because the redesigned interface prevents incomplete sortings * Unskip tests * Remove post m card cell * Remove reference to redesign from test * Remove comment m card * Unskip accessibility test in debates versions * Remove deprecated test * Remove deprecated comments about redesign * Remove deprecated test The proposals card-l does not show the supports count in the metadata, so there is no difference when supports are enabled on proposals component or not * Remove useless reference to redesign in test * Remove REDESIGN_PENDING comment: do not use card-l cell for media links * Remove REDESIGN_PENDING comments * Remove unused partial * Unskip tests * Remove conference m card cell * Adapt authorizations tests to redesign * Remove deprecated tests * Avoid errors in upload modal when file validation fails * Unskip tests * Remove deprecated test * Remove REDESIGN_PENDING comments * Update map styles in proposals * Uncomment tests * Uncomment and adapt selector in tests * Remove reference to redesign * Fix linter offense * Remove obsolete brakeman skip and update other in proposals * Remove unused translations and add some wizards keys to ignore unused * Update expected url in activity cell with comments activities * Increase header levels in version cell to include h1 * Recover missing data attribute in hero content block * Fix coordinates update in add_proposal js * Hide map only if there is no address with coordinates * Empty address field before changing it in test * Add issue reference to skipped tests * Display replies on deleted comments and keep then with AJAX deletion refresh * Unskip test * Update decidim-core/app/cells/decidim/version/show.erb Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Use h2 tag * Move shared examples require to core * Replace actions with method doing the same * Fix test * Remove references to redesign in shared examples --------- Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Redesign: refactor the scope of the main Javascript initializer (decidim#11317) * set the scope of all functions that are initialized at startup * scope the datepicker as well * remove precated js stuff (testing) * fix lint & place accordingly the initializations * Redesign: deprecate foundation-datepicker from public site (decidim#11420) * deprecate foundation-datepicker public site * fix core tests * fix datepicker tests * fix lint and rubocop * replace now with current * fix tests * fix more tests * fix tests * remove byebugs * Redesign: don't submit forms on modal's cancel * Update documentation for here map configuration (decidim#11447) * Two-columns layout fix (decidim#11400) * Column conversations fix * suggested changes added to &__aside * &__main updated * Changes made to span on &__main * test * spacing cli test * Disabler for pointer events (decidim#11449) * Added a disabler for pointer event * Improved workaround for the tabs +1 * Indentation line 19 * Fix test component (decidim#11436) * Fix label position on report button (decidim#11435) * Fix pipeline after decidim#11449 (decidim#11455) * Upgrade postgres from 11 to 14 (decidim#11453) * Fix initiatives ransack filters (decidim#11329) * Revert "Lock ChromeDriver to the latest working version (decidim#11391)" (decidim#11458) This reverts commit 2450714. * Force capital starting letter for PR titles (decidim#11457) * Force capital starting letter for PR titles * Force the starting character to be capitalized * Allow disabling password expiration (decidim#11135) * Allow disabling password expiration NIST does not recommend expiring passwords: https://pages.nist.gov/800-63-FAQ/#q-b05 * Add documentation about the new option * Fix linter * Fix linter * Fix autocomplete result list duplicates (decidim#11442) * Fix autocomplete user group duplication / Add system test * Fix test * Redesign: organization colors (decidim#11451) * refactor the organization colors * typo * remove specs about them_color input * fix specs * use old red for testing * set the default redesign colors to the organization factory (color-contrast) * replace the uses of legacy primary color * Redesign: axe accessibility issues (decidim#11403) * dropdown card does not require an id * add aria-label to progress bar * disable dropdown for desktop navigation * Refactor the use of filter_text_for to be in views instead of helpers * Allow passing an id to filter_text_for method * remove element_id (testing) * pass the id to the label * distinguish filters and order controls * distinguish all uses of dropdown-menus * rename control * don't use a list when you need divs * disable dropdown for desktop navigation * update code for emoji * remove ids from tests * fix core tests * fix election tests * fix tests * remove profile__user-role class * remove needless aria-attribute * remove duplicated ids * empty merge * Include alternative title for images when there's nothing but the title * disable dropdown for desktop * add alt text to default image * user profile must not be an h5 * duplicate_id and copy button SR text * fix odd layout for edit initiative * redesign committee requests form * fix a11y issue on budgets project list * mark layout item header al role=complementary since it's outside * do not expand elements if the page has very few contents * fix a11y issue on debates, sortitions and forms * fix rubocop * fix erblint * use the units as the aria-label * standarize arias for progress bars * initiatives style glitch --------- Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com> * Integrate Bullet in decidim-dev (decidim#11416) * Initial commit on bullet integration * remove uneeded condition * Disable failing actions * Disable failing workflows * Disable all the workflows * Refactor actions * Revert generator change * Redesign: comment button (decidim#11461) * render the comment button depending of the settings * enhance condition: initiatives is not a component * handle comments_blocked? in case of null * fix offenses * refactor duplicated views (decidim#11459) * Redesign: remove the item itself from nav menu (decidim#11452) * remove the item itself from nav menu * fix i18n * fix specs * fix specs * Refactor Decidim jobs to inherit from Decidim::ApplicationJob (decidim#11468) * overwrite dropdown defaults for tablet size (decidim#11479) * Fix ambiguous id column on projects query (decidim#11474) * Redesign: resource metadata (decidim#11463) * make every resource metadata equal * remove any reference to "drawer" * typo * slightly different initiative layout * Redesign: display alert when no projects (decidim#11480) * display alert when no projects * normalize locals * Redesign: system (decidim#11478) * prepare assets and login page * responsive sidebar system * organization forms * refactor organization forms * password forms * admins forms * oauth forms * fix typo * fix specs and required update modal * fix lint * fix lint * update test settings * avoid aside to expand * remove deprecated styles * Fix persistance of the area in initiative creation (decidim#11476) * Lock TipTap editor to ~2.0.3 (decidim#11492) * Redesign: consolidate 2-col layout (decidim#11426) * spacing for participatory_processes, proposals, collaborative drafts, participatory texts and component announcement * spacing for sortitions, elections, meetings and fixes over proposals * spacing for assemblies and fixes over participatory spaces * fix tests * spacing for core * spacing for conferences, blogs and accountability. Fixes over conversations * aside heading responsive and spacig for budgets * spacing for projects, debates and initiatives * fix initiative specs * fix proposals specs * fix specs * missing space for collaborative drafts * avoid overwritting css rule du to tailwind directive * missing class * avoid weird space around the span * missing condition * Encode non-ASCII characters on external links (decidim#11472) * Encode non-ASCII characters on external links * Lint * Apply suggestions * Fix weird behavior with votes in comments (decidim#11490) * Fix weird behavior with votes in comments * Fix flaky test * Update remixicon to 3.5.0 (decidim#11494) * Replace close text with icon in order to enlarge it (decidim#11487) * Remove deprecated views & translations (decidim#11488) * Incorrect component link generation (decidim#11432) * Fix SEO routing issue for components index * Add tests for the redirects * Fix failing tests * Fix failing specs * Fix all failing tests * Fix linting error * default gray (non-text contrast) (decidim#11502) * Highlight the toggle contrast (decidim#11506) * Allow tabbing on mobile devices (decidim#11501) * Redesign: finish create-initiative step (decidim#11493) * simplify the finish create-initiative step * restore original condition * Fix the length of edit button in proposals (decidim#11497) * Fix wrong 'no activity' text on last activities page (decidim#11496) * Fix wrong 'no activity' text on last activities page * Fix failing spec * Redesign: meetings code icon (decidim#11486) * replace qr code icon with another related * change icon * Redesign: highlight links on hover (decidim#11508) * hightlight links on hover * highlight footer links * distinguisable link banner * place "skip to content" the first thing of all * Focus styles (decidim#11509) * Redesign: semantic datetimes (decidim#11513) * include datetime semantically * calendar a11y * ignore missing translation * Make visible the skip map option (decidim#11515) * Render partner without any link if it's not configured (decidim#11512) * Show different text for the Register CTA when the user is already registered (decidim#11511) * Show different text for the Register CTA when the user is already registered * Change two more buttons * Normalize i18n * Don't show initiative state field instead of showing it disabled (decidim#11518) * Don't show initiative state field instead of showing it disabled * Don't show the field in the edit form neither * Add tests * Lint files * Update decidim-initiatives/app/views/decidim/initiatives/initiatives/_form.html.erb Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> --------- Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Display special characters in breadcrumb menu (decidim#11514) --------- Co-authored-by: Hugoren Martinako <aumpfbahn@gmail.com> Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> Co-authored-by: Ruth Cheesley <ruth@ruthcheesley.co.uk> Co-authored-by: Eduardo Martínez <eduardomech@gmail.com> Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com> Co-authored-by: Tom <101816158+greenwoodt@users.noreply.github.com> Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi> Co-authored-by: Carlo Beltrame <carlo@beltra.me> Co-authored-by: JoonasAapro <110532525+JoonasAapro@users.noreply.github.com> Co-authored-by: Fran Bolívar <francisco.bolivar@nazaries.com>
andreslucena
added a commit
that referenced
this pull request
Oct 16, 2024
* Add events for comments * Add events for debates * Add events for meetings * Update the proposals commands * Refactor with_events * Apply review recommendations * Create decidim-ai module Add Gitlab action workflow Patch the generator Running linters Gemfiles * change description * Add language detection-service Add language service Normalize gems * Add registry strategy (#253) * Add SpamDetectionService class (#255) * Add BayesStrategy (#256) * Add BayesStrategy * Add Bayes Analyzer * Refactor strategy intialization process * Change the pipeline working dir * Fixing spam suite * Revert event changes * Revert event changes * Add BayesStrategy (#256) (#257) * Add BayesStrategy * Add Bayes Analyzer * Refactor strategy intialization process * Running linters * Add score calculation (#262) * Add event handlers and spec data (#263) * Add event handlers and spec data * Fixing failng specs * Fix Catgeory error in untrain * fix decidim ai tests * Refactor AI namespaces (#269) * Add Strategy module * Add more namespaces * Add resources to be indexed (#254) * Add resources to be analyzed * mend * Add action to reset train model (#270) * Add documentation page (#275) * Add Initiatives to antispam (#271) * Redesign: rename tests (#11184) * Remove skip_unless_redesing_enabled calls in tests * Remove redesign_helpers for tests * Unskip and adapt REDESIGN_PENDING examples * Remove deprecated examples * Allow to configure the default header level of attributes titles in diff cell * Ensure accessibility of h in collaborative draft show * Recover accessibility tests in proposals * Remove comments * Unskip and adapt test * Fix expected text of link in test * Remove check for removed follow actions in participatory text index * Unskip tests * Unskip and adapt to vote proposal tests * Remove deprecated methods * Remove deprecations in tests * Remove collaborative drafts and proposals deprecated m cells * Adapt proposal m cell cache hash to l cell * Adapt proposal m-cell specs to l-cell * Replace use of proposal_m card * Remove condition on redesign from test * Rename shared examples for attachments to not use redesign term * Remove tests for CTA behavior deprecated in processes card-g * Avoid duplicated ids of highlighted participatory process in menu breadcrumb * Recover skipped examples * Recover accessibility tests in participatory processes * Remove participatory process description specs and include expected contents with metadata content block enabled * Unskip REDESIGN_PENDING tests * Move common statistics cells tests to shared examples * Remove unused process metadata item * Remove deprecated proceses m card and add tests for l and g cards * Remove deprecated process group m card cell * Recover tests after redesign in initiatives * Remove skipped deprecated tests * Remove assemblies description page test and move examples to show page with appropriate content blocks * Remove assembly m card and related tests * Unskip tests after completing redesing * Adapt sorting test to redesigned interface * Remove deprecated sorting tests because the redesigned interface prevents incomplete sortings * Unskip tests * Remove post m card cell * Remove reference to redesign from test * Remove comment m card * Unskip accessibility test in debates versions * Remove deprecated test * Remove deprecated comments about redesign * Remove deprecated test The proposals card-l does not show the supports count in the metadata, so there is no difference when supports are enabled on proposals component or not * Remove useless reference to redesign in test * Remove REDESIGN_PENDING comment: do not use card-l cell for media links * Remove REDESIGN_PENDING comments * Remove unused partial * Unskip tests * Remove conference m card cell * Adapt authorizations tests to redesign * Remove deprecated tests * Avoid errors in upload modal when file validation fails * Unskip tests * Remove deprecated test * Remove REDESIGN_PENDING comments * Update map styles in proposals * Uncomment tests * Uncomment and adapt selector in tests * Remove reference to redesign * Fix linter offense * Remove obsolete brakeman skip and update other in proposals * Remove unused translations and add some wizards keys to ignore unused * Update expected url in activity cell with comments activities * Increase header levels in version cell to include h1 * Recover missing data attribute in hero content block * Fix coordinates update in add_proposal js * Hide map only if there is no address with coordinates * Empty address field before changing it in test * Add issue reference to skipped tests * Display replies on deleted comments and keep then with AJAX deletion refresh * Unskip test * Update decidim-core/app/cells/decidim/version/show.erb Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Use h2 tag * Move shared examples require to core * Replace actions with method doing the same * Fix test * Remove references to redesign in shared examples --------- Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Redesign: refactor the scope of the main Javascript initializer (#11317) * set the scope of all functions that are initialized at startup * scope the datepicker as well * remove precated js stuff (testing) * fix lint & place accordingly the initializations * Redesign: deprecate foundation-datepicker from public site (#11420) * deprecate foundation-datepicker public site * fix core tests * fix datepicker tests * fix lint and rubocop * replace now with current * fix tests * fix more tests * fix tests * remove byebugs * Redesign: don't submit forms on modal's cancel * Update documentation for here map configuration (#11447) * Two-columns layout fix (#11400) * Column conversations fix * suggested changes added to &__aside * &__main updated * Changes made to span on &__main * test * spacing cli test * Disabler for pointer events (#11449) * Added a disabler for pointer event * Improved workaround for the tabs +1 * Indentation line 19 * Fix test component (#11436) * Fix label position on report button (#11435) * Fix pipeline after #11449 (#11455) * Upgrade postgres from 11 to 14 (#11453) * Fix initiatives ransack filters (#11329) * Revert "Lock ChromeDriver to the latest working version (#11391)" (#11458) This reverts commit 2450714. * Force capital starting letter for PR titles (#11457) * Force capital starting letter for PR titles * Force the starting character to be capitalized * Allow disabling password expiration (#11135) * Allow disabling password expiration NIST does not recommend expiring passwords: https://pages.nist.gov/800-63-FAQ/#q-b05 * Add documentation about the new option * Fix linter * Fix linter * Fix autocomplete result list duplicates (#11442) * Fix autocomplete user group duplication / Add system test * Fix test * Redesign: organization colors (#11451) * refactor the organization colors * typo * remove specs about them_color input * fix specs * use old red for testing * set the default redesign colors to the organization factory (color-contrast) * replace the uses of legacy primary color * Redesign: axe accessibility issues (#11403) * dropdown card does not require an id * add aria-label to progress bar * disable dropdown for desktop navigation * Refactor the use of filter_text_for to be in views instead of helpers * Allow passing an id to filter_text_for method * remove element_id (testing) * pass the id to the label * distinguish filters and order controls * distinguish all uses of dropdown-menus * rename control * don't use a list when you need divs * disable dropdown for desktop navigation * update code for emoji * remove ids from tests * fix core tests * fix election tests * fix tests * remove profile__user-role class * remove needless aria-attribute * remove duplicated ids * empty merge * Include alternative title for images when there's nothing but the title * disable dropdown for desktop * add alt text to default image * user profile must not be an h5 * duplicate_id and copy button SR text * fix odd layout for edit initiative * redesign committee requests form * fix a11y issue on budgets project list * mark layout item header al role=complementary since it's outside * do not expand elements if the page has very few contents * fix a11y issue on debates, sortitions and forms * fix rubocop * fix erblint * use the units as the aria-label * standarize arias for progress bars * initiatives style glitch --------- Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com> * Integrate Bullet in decidim-dev (#11416) * Initial commit on bullet integration * remove uneeded condition * Disable failing actions * Disable failing workflows * Disable all the workflows * Refactor actions * Revert generator change * Redesign: comment button (#11461) * render the comment button depending of the settings * enhance condition: initiatives is not a component * handle comments_blocked? in case of null * fix offenses * refactor duplicated views (#11459) * Redesign: remove the item itself from nav menu (#11452) * remove the item itself from nav menu * fix i18n * fix specs * fix specs * Refactor Decidim jobs to inherit from Decidim::ApplicationJob (#11468) * Add Initiatives to antispam * Add initiatives specs --------- Co-authored-by: Eduardo Martínez <eduardomech@gmail.com> Co-authored-by: Hugoren Martinako <aumpfbahn@gmail.com> Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com> Co-authored-by: Tom <101816158+greenwoodt@users.noreply.github.com> Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi> Co-authored-by: Carlo Beltrame <carlo@beltra.me> Co-authored-by: JoonasAapro <110532525+JoonasAapro@users.noreply.github.com> * Fix rubocop * Running linters * FIx failing spec * Gem revert * Fix gem dependencies * Fix test suite * Fix spec * Revert wicked_pdf version change * Add spam csv dictionaries to the list of exceptions * Additional CSV * Spell checks * Spell checks * Add Autolabeler config for Ai module * Fix specs * Refactor Language Detection * Refactor models * Refactor reporting_user_email setting * Add modules for jobs * Refactor reporting user * Refactor trained models * Add user_classifier refactor * Rename LanguageDetection ruby module * Addded the text cleanup class * Add some documentation * Add ngrams to Spelling checks * Fix rubocop * Fix Gemfiles * Update decidim-ai/README.md Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi> * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Fix more issues from code review * Add module documentation page structure * Add initializer template * Refactor language engine * Add checks for classifier contract * Fix pipeline * Fix errors * Fix the checkspel * Fix the checkspeller * Move the backed to Redis as default * More fixes * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Add doc links * Fix core spec * Fix the AI classifier * Fix the specs * Add safe navigation * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Fix namespace * Update install * Add fallback connection * Lints * Update decidim-ai/README.md Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Cleanup * ENV vars * More ENV * Apply suggestions from code review Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Remove host param from url uploader method on Conferences (#13508) * Lock Ubuntu to 22.04 in github actions (#13513) * fix translation bar css (#13512) * Add Linked Meetings feature to allow linking meetings to other spaces (#13169) * Add linked meetings feature to allow linking meetings to other participatory spaces (MeetingLink model) * Update meeting components selctor to not show the current_component and show non published components * Fix rubocop offenses * Fix eslint offenses * Fix erblint offenses * Fix spelling linter * Normalise locales * Fix several admin/meetings specs * Fix specs * Add specs * Fix rubocop offenses * Disable assign button with private non transparent meeting * Add tooltip to unlink action button * Update meeting redirection message * Fix linking a meeting to private and transparent spaces * Fix unlink tooltip in dynamic elements * Normalise locales * Fix redirect callout, avoid using hash syntax * Move redirect notice to the frontend * Fix meeting cell when shown outside of a component * Escape space names * Fix meeting link in accountability results * Fix meeting cell when url extra params are passed * Unify private_non_transparent logic * Fix rubocop offense * Fix method * Add specs for private_non_transparent_space? method in component * Create a content block for open debates on the home of a space (#13510) * add content blocks for debates * fix debetes order * add specs * change HighlightedDebatesCell * change HighlightedDebatesCell * refactor HighlightedDebatesCell * Update decidim-debates/spec/cells/decidim/debates/content_blocks/highlighted_debates_cell_spec.rb Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * fix test --------- Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro> * Remove CLD * Add User group tests * More review * more review * Remove module dataset training * Update docs/modules/develop/pages/ai_tools.adoc Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> * Remove doc bits * Apply suggestions from code review --------- Co-authored-by: Eduardo Martínez <eduardomech@gmail.com> Co-authored-by: Hugoren Martinako <aumpfbahn@gmail.com> Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com> Co-authored-by: Tom <101816158+greenwoodt@users.noreply.github.com> Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi> Co-authored-by: Carlo Beltrame <carlo@beltra.me> Co-authored-by: JoonasAapro <110532525+JoonasAapro@users.noreply.github.com> Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com> Co-authored-by: Ivan Vergés <ivan@pokecode.net> Co-authored-by: Màxim Colls <collsmaxim@gmail.com> Co-authored-by: Anna Topalidi <60363870+antopalidi@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎩 What? Why?
Solve a set of accessibility issues that AXE detected
Reviewed pages: https://docs.google.com/spreadsheets/d/1aszjT6QC3sPwUb6pRT3emRs4-5kpkkuVQrZS78YS6_0/edit#gid=758727657