Skip to content

Redesign: accountability#9772

Merged
ahukkanen merged 121 commits intodevelopfrom
feature/redesign-accountability
Nov 8, 2022
Merged

Redesign: accountability#9772
ahukkanen merged 121 commits intodevelopfrom
feature/redesign-accountability

Conversation

@Crashillo
Copy link
Copy Markdown
Contributor

@Crashillo Crashillo commented Aug 31, 2022

🎩 What? Why?

New screens for Accountability module

There are some features shared with other components like versions, comments and attachments that are not yet with the new designs. This will be done in future PRs

📷 Screenshots

(user: decidim / pwd: staging123)

♥️ Thank you!

1 Turbo is yet not integrated, so the functioning (mainly the way to trigger the content) is pretended

@Crashillo Crashillo added the project: redesign Barcelona City Council contract label Aug 31, 2022
@Crashillo Crashillo marked this pull request as ready for review September 9, 2022 09:32
@entantoencuanto entantoencuanto force-pushed the feature/redesign-accountability branch 2 times, most recently from f2c88e6 to 4dba253 Compare September 16, 2022 18:15
@Crashillo Crashillo mentioned this pull request Sep 21, 2022
@Crashillo Crashillo force-pushed the feature/redesign-accountability branch from 6d14893 to 3db68e9 Compare September 21, 2022 12:32
@entantoencuanto entantoencuanto force-pushed the feature/redesign-accountability branch from d50cd83 to a8b346b Compare September 29, 2022 11:04
@entantoencuanto
Copy link
Copy Markdown
Contributor

Hi, @ahukkanen, @decidim/product, this PR is ready for review

There are some features shared with other components like versions, comments and attachments that are not yet with the new designs. This will be done in future PRs

@andreslucena
Copy link
Copy Markdown
Member

Hi @entantoencuanto @Crashillo
I'm a bit lost on how this is handled usually, as I was disconnected with the review process, but I see that according to the Project, the following step should be an "Internal review" made by you guys, right?
image

@Crashillo Crashillo requested review from a team and ahukkanen September 29, 2022 14:21
@Crashillo
Copy link
Copy Markdown
Contributor Author

Yep, that's right, internal review it's a column for us, usually @rober-gd, to do a first review about the styling, but it could arrive beyond that. Sometimes, there no need to go column by column (for whatever reason), so we can skip it.

When all things have been done in our side, we move it to the Ready column, but, most importantly, we request a review directly from you all.

@andreslucena
Copy link
Copy Markdown
Member

Sometimes, there no need to go column by column (for whatever reason), so we can skip it.

I don't understand this. Are you telling there's a reason for not to follow the process in this particular PR? Honestly, for what I've seen, the reviews made by @rober-gd are truly great and if he doesn't do that, it seems like it's extra work that me (or anyone else from @decidim/product) need to do.

@Crashillo
Copy link
Copy Markdown
Contributor Author

Crashillo commented Sep 30, 2022

Regarding the PR. "Our" columns are: frontend (integrating front, @Crashillo), backend (integrating back, @entantoencuanto), review (internal review @rober-gd). Once done, we move it to Ready, where @ahukkanen does a final review after all tests are green (or most of them).

This is the normal workflow, but it could be a PR would have not frontend (therefore no design review) or not backend, in such cases we skip some columns and move it to the corresponding column.

In any case, every single PR finally goes to Ready, where it holds there until merge. Since last product meeting we were requested to include @decidim/product as reviewers, so we did. Is this not what you would like? In such case we take you off from reviewers, as you prefer the most.

@ahukkanen
Copy link
Copy Markdown
Contributor

@entantoencuanto @Crashillo

I'm having a bit of hard time reviewing this. I don't understand the views at all but it may be just a problem with the structure of the accountability module as it is today because I don't even completely understand how it works in the old design.

I checked both the actual UI as well as the design app but neither of those make any sense to me.

So could you help me a bit with this:

  1. Could you share the UI comps for this, so I could compare with the actual functionality?
  2. Could you provide a bit of explanation for the views? Particularly the "Show" view that only has a "Trigger" button on it, I understand this trigger is supposed to be launched somewhere else and it's there just as an example how it would appear. Any help here would be appreciated.

@ahukkanen
Copy link
Copy Markdown
Contributor

Code-wise, could you also check that the sw.js and sw.js.map don't get to the repository as they are currently part of this PR marked to be merged. They are causing few issues at CodeClimate but they should be ignored, as you've properly marked them in the .gitignore.

@Crashillo
Copy link
Copy Markdown
Contributor Author

Particularly the "Show" view that only has a "Trigger" button on it, I understand this trigger is supposed to be launched somewhere else and it's there just as an example how it would appear. Any help here would be appreciated.

As I pointed in the OP, that view is partly mocked. It requires Turbo to work properly... problem is Turbo is not yet ready (@entantoencuanto is working on that somewhere else). Since Turbo integration is far from straight-forward, what we have done here is a mock for pretending the actual feature, to avoid mix in this PR accountability stuff with Turbo (concerns separation).

The "show" view will be within the "index" view, but in the meantime, we fake it as different views. You can see in the proto how will look like once integrated. This thing is tracked in #9775

Copy link
Copy Markdown
Contributor

@ahukkanen ahukkanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly looks fine, just few mobile issues and a couple of general structural issues.

There are still many things pending in this but I guess that's the plan here as we discussed before.

@carolromero
Copy link
Copy Markdown
Member

carolromero commented Oct 24, 2022

@Crashillo from the @decidim/product side looks quite good! Some details to check before we move forward:

  • I realize that we're using the same icon for scopes and categories, and this can be misleading. I actually got confused when filtering the results. We may need to find a different icon for the scopes cc/ @rober-gd.

  • The color of the percentages does not match. It looks like some opacity attribute for the text is missing.
    image

  • Can we use an svg for displaying a date range?
    image

ahukkanen
ahukkanen previously approved these changes Oct 25, 2022
Copy link
Copy Markdown
Contributor

@ahukkanen ahukkanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From technical perspective it seems fine now if you can fix those issues reported above.

Just as a general comment, I think the drawer is generally a bad idea for content discoverability. With the drawer, I believe we won't have individual URLs to the results right? This makes it really hard to make the content discoverable outside of Decidim.

On the other hand, if we keep the individual URLs for the results, it makes the problem smaller but then we have many different URLs with the exact same <h1> title (the component name). Each individual page should have an individual <h1> on it which would indicate the main content on that page, e.g. the title of the result.

If you make the result title a <h1> with the drawer, you will introduce another accessibility issue as then you would have two <h1> elements on the same page.

Other than that rant (and my previous rant about the UI complexity of this component), I think this is ready to go if you fix the issues reported above.

@entantoencuanto
Copy link
Copy Markdown
Contributor

The idea of the drawer is to use turbo to update the URL once open. I've been testing this in this branch. When the drawer is open the URL is updated and once closed the original one is recovered (however there are still problems recovering the path in some situations, for example when closing by pressing the esc key)

@ahukkanen
Copy link
Copy Markdown
Contributor

OK, thanks for the explanation.

Then the second issue still persists: we will have multiple different URLs on the site with the exact same <h1>. This is not good for semantics or blind users.

@Crashillo
Copy link
Copy Markdown
Contributor Author

May we go forward with this PR?

@entantoencuanto
Copy link
Copy Markdown
Contributor

The drawer feature using turbo is currently in process in #9884. This includes the dynamic replacement of h1 elements when a drawer is opened from the index. This PR should be merged first in order to advance in the other one

Copy link
Copy Markdown
Member

@carolromero carolromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good from Product side

@carolromero
Copy link
Copy Markdown
Member

I just realized that is not done as asked
image

@Crashillo can you please review it again?

Copy link
Copy Markdown
Member

@carolromero carolromero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see my last comment

@Crashillo
Copy link
Copy Markdown
Contributor Author

@Crashillo can you please review it again?

Perhaps you're seeing some cached web? That's how it looks like now:
imagen
https://decidim-redesign.populate.tools/processes/smile-cutting/f/16/results

@carolromero
Copy link
Copy Markdown
Member

Perhaps you're seeing some cached web?

yeah I hate to be the nitpicky one (and you probably hate it too), but there is an arrow in the design:
image

@Crashillo
Copy link
Copy Markdown
Contributor Author

Crashillo commented Nov 3, 2022

Ah ok, I see now the misunderstanding, you were referring to the design proto, I was saying that the previous arrow -> character was already uploaded into a simple dash - in the staging site.

We've agreed to use that, instead of the design arrow, due to both dates come from the backend in a single pack, therefore adding an icon arrow inbetween were overcomplicated, besides creating an extra css issue. Is that OK? Our bad, design proto wasn't uploaded accordingly.

@Crashillo
Copy link
Copy Markdown
Contributor Author

Coverage is not passing, but all tests are green, may we go forward on this? It's important due to css components 🙏

@carolromero
Copy link
Copy Markdown
Member

@Crashillo green light from @decidim/product

Copy link
Copy Markdown
Contributor

@ahukkanen ahukkanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works OK apart from the drawer aspect which I think is a really bad idea for semantics generally.

I just noticed this mobile layout issue with the tags and if the result has sub-results it seems a bit cluttered on the mobile screen:
Cluttered mobile screen

I will move this issue to #9775.

The search section does not have a "search" button which is an accessibility violation:
image

I will move this issue to #9775.

The header menu is also broken on mobile:
Broken header menu

I will move this issue to #9669.

All in all, as I've mentioned before, I still find the UX confusing with the accountability module but this is more related to the overall structure of the module if we want to match the current functionality in Decidim. This could use some re-thinking in the future how this module works, it's currently very complex and hard to use (not because of the redesign efforts, just because of the UX of this module).

@ahukkanen ahukkanen merged commit 002e1d5 into develop Nov 8, 2022
@ahukkanen ahukkanen deleted the feature/redesign-accountability branch November 8, 2022 13:29
Crashillo added a commit that referenced this pull request Nov 10, 2022
* home accountability desktop

* extract filters to component

* index accountability desktop

* add headers & updated sidebar widget

* refactor status, mark as deprecated

* extract css components

* generalize drawer

* use all the core assets from app-design

* larger scope for stylelint overrides in gem stylesheets (not only core)

* missing semicolon stylelint

* home acc. mobile design

* filters redesigned for mobile (also profile)

* update dropdown/filters component

* update filters in user_activity

* enhance auto-close for dropdowns

* fix erblint

* move drawer to normal app

* refactor list of projects

* move generic metadata to css component

* remove deprecated

* cancel out padding right on reverse layouts

* Fix global moderation types not translated (#9662)

* Fix unpublished processes shown in the group process count (#9682)

* Fix a typo in the logic customization manual (#9831)

* Extract status to a cell

* Move filter items calculation to a helper method

* Fix test to not use removed stylesheets imports

* Update selector in test

* Fix tags selectors in tests

* Fix linter offenses

* Remove unused translations

* Normalize translations file

* add inner-button default styles

* remove CSS duplicated as of 695e68a

* extract css filters

* Define a cell to display results data

* Define a category item in result cell

* Define a dates_item_compact element in result cell

* Add a variant of project cell with different template and items

* Allow status cell to accept models different to categories

* Fix condition to build items array

* Extend the use of status cell

* Define linked_resources_for at cell level

* Refactor results and projects views to use cells

* Remove unused partials

* Remove unecessary line in push notifications spec (#9833)

* Fix updating organization settings in case there were errors (#9634)

* Fix source format in the data consent docs for the example (#9837)

* Redesign: conversations (#9469)

* conversations index

* conversation show

* reformat scroll to bottom no jquery

* fix naming

* allow only one initialization on load

* scroll to last child fix on reloads

* conversations shows + first approach form validation no-abide

* Recover form-error class in error messages in forms

* Fix translations

* Call author cell with context actions

* Replace TODOs with REDESIGN_PENDING

* Fix rubocop offense

* Refactor conversations to allow creation

* Recover jQuery use

* Update test after abide removal

* Fix tests

* Allow set novalidate in decidim_form_for helper method

* Recover novalidate to true by default in decidim_form_for

* Move messages to controller and remove duplicated @Conversations

* Validate initiative fill_data form in front

* Fix typo and add validation_messages as helper_method

* extract conversation styles

* update markup

* generalization form error

* new redesign conversations

* new index conversations

* generalization character counter

* input moji feature

* fix messages layout

* fix wrapper char counter

* fix style issues

* update conversations/new mockup

* responsive aside adaptations

* aside buttons conversations

* reorder time in conversations

* unread condition

* ignore stylelint

* mobile conversations

* position emoji

* fix position helper messages on generic forms

* add character counter fix

* Recover abide in forms

* Fix reply js replacement after creating a conversation

* Remove unused translations

* Fix recovering of abide in forms

* Fix selectors in tests

* Recover old modal behavior

* Update expected texts in tests

* Hide test that may no longer be needed

* Update selector in test

* Update selectors in test

* Comment maybe deprecated example

* Remove redesign pending comments

* remove redupe dep

* Force id of new conversation modal form

* Replace main with div in wrapper layout part

This avoids accessibility errors testing redesigned parts using old
layouts

* tune aside layout for allow different setups in mobile

* Use an identifier for main section to be used in tests

* Update selector in tests looking for main

* Recover main element in legacy wrapper

* add css classes to shrink icon size

* fix desktop sidebar layout

* Extract display_name method in author cell

* Hide messages button in profile actions when user has muted conversations

* Display unread pending indicator in redesigned layout main links when there are unread conversations also

* Enable full redesign feature in conversations integration tests

* Fix tests of pending conversations

* Update tests with redesigned author cell

* Update tests of messages in profile

* Skip temporarily accessibility tests in conversations due to pending redesign issues

* Skip test pending of another PR merge to pass

* Recover skipped test

* Fix footer h sequence

* Fix not allowed “no-external-link” on element “a” accessibility error

* Replace class to hide dataconsent-warning element

* Replace time with invalid value format with div

* Replace invalid div inside summary with span

* Add alt to image tag

* Fix h sequence

* Avoid having too much elements inside label tag

* Remove repeated line

* Recover accessibility tests in conversations

* Recover move hidden class to attribute in dataconsent-warning element and fix javascript

* Recover hide class for compatibility with old design

* Change tag in two cols layout to ensure accessibility with old design

* Add extra check

* Define set_redesign using environment

* Attempt to enable redesign in tests

* Attempt to enable redesign in tests using configuration persisted in database

* Change primary color to improve contrast

* Force new commit

* Force redesign_active

* Redefine primary color

* Disable tweak_bootsnap

* Allow forcing redesign in controllers

* Adapt test to parts with redesign not enabled

* Revert "Force redesign_active"

This reverts commit 556172c.

* modal required files

* conversations modal and redesigned_icon jquery-free 🎉

* Replace div tag with span to ensure accesibility

* Replace div tag with span to ensure accesibility

* Fix linter offense

* Remove unused translation

* Revert "Disable tweak_bootsnap"

This reverts commit b23a400.

* Prepare follow feature related tests to both redesigned and not redesigned versions

* missing omniauth buttons

* fix initializer multiple-mentions

* fix autocomplete styles

* add disabled for buttons

* Add missing id

* Disable automatic id

* Remove deprecated tests

* Skip test using old modal inside a page that will be removed

* Recover hide class in condition for compatibility with old design

* Remove commented code

* Fix announce threshold calculation bug

* Move reply form to a partial and reuse it

* add i18n button title

* remove autofocus

* alter order buttons + shrink size

* move styles to file

* Fix translation alphabetical order

* reduce size button modal & align center sidebar buttons

* order aside actions

* add border transparent to default button

Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com>

* Fix i18n-tasks issues

* move filter search to css component

* restore search element

* Define a new kind of card to use for component resources in lists and create one for accountability results

* Update test

* Add metadata cell, use it in card_l and refactor some parts

* Refactor results card_l using metadata

* Update styles

* Recover drawer layout

* Update categories hierarchy display in filtered projects indexes

* Update the CHANGELOG template and fix detailed changes heading levels (#9843)

* Update the changelog template and fix detailed changes heading levels

* Make the same updates to the current CHANGELOG

* Fix the heading level for the API change sub sections

* Add tailwind as an API change

* Fix typo

* Fix moderations for comments that are mapped to deleted resources (#9621)

* Return nil for reported content url if the commentable is deleted

* Show "deleted resource" instead of the visit URL when URL is not returned

* Add system specs testing that moderations for comments with deleted resources work

* Fix issue hiding comments for deleted debates

* Add moderation test to hide comments for deleted resources

* Add manage moderations shared examples for debates

* Fix specs after merging with develop

* Do not import resources multiple times (#9644)

* Fix form answer attachments breaking the answer view (#9653)

* Improve release process (#9492)

* Improve the release related comands

- Abort the process if there are uncommitted changes
- Abort the process in case the official locales are not complete
  or clean
- Do not release NPM packages in case some of the gem releases
  fail
- Add a feature to retry the commands and make it configurable
- Enable retrying the commands by default assuming using OTP
  authentication

* Disable the GemManager retries for the generators specs

* Configure retry times env variable not to retry infinitely

* Configure the RETRY_TIMES env var for the gem installations in spec

* Move the RETRY_TIMES configuration to the spec helper

* Rubocop

* Fix user related absolute URLs (#9596)

* Fix the user related URLs to contain the correct port

* Add a spec for the profile_url method in UserPresenter

* Fix default_avatar_url in UserPresenter

* Fix specs broken by the port change

* Fix broken specs

* Call the same method in both cases

* Fix "disappearing" underscores with comments (#9654)

The underscores were actually converted into `<em>`s which is not
desired.

* Fix editor content saving when the content has only one video (#9629)

* Allow only videos on the rich text editor

* Add a spec testing adding only a video through Quill

* Fix date/time formats at component forms (#9632)

* Fix the localized date and time with zone with ISO formats

* Fix the datetime_field with Time and DateTime

* Change the component time to TimeWithZone attribute type

* Fix the conferences admin update controller spec

* Fix last activity page showing recently updated records (#9844)

* Fix last activity page showing recently updated records

* Fix missing verb

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Fix collaborative drafts page when there are errors on the form (#9656)

* Fix the finite value on the debate form when editing an existing debate (#9645)

* Fix importing participatory process from legacy format (#9646)

* Fix exporting and importing a page component (#9638)

* Fix webpacker crashes on missing icons (#9636)

* Remove GIF and BMP from the default file formats (#9641)

* Remove gif and bmp from the default allowed file formats

* Remove gif from the core examples

* Add results metadata in show footer

* Fix argument in meetings panel

* Hide progress when not present in status cell

* Allow displaying status on results breadcrumb even if there are no results

* Display attachments in result show (resesign pending)

* Hide scope filter when component has no subscopes

* hide global status from home if progress disabled in component

* Update selector to work in redesigned parts

* Skip test pending of redesign

* Change selectors in tests

* Remove not expected result in test

* Add interaction to tests

* Update expected content in test

* Skip comments tests pending of redesign

* Fix selector in test

* add CSS component drawer_metadata

* Remove unused argument of helper method

* Add missing semicolon in scss file

* Fix linter offense

* Add documentation page for accessibility tips (#9608)

* Add page for accessibility tips

* Add the example for accessible icon links

* Typo fixes and refinements

* Refer to the accessibility guide from the styles customization

* Document common accessibility mistakes

* Valid example HTML

* Generalize list styling

* Add tips about headings

* Fix typo

* Update the page with the points from #6207

* Apply suggestions from code review

Fixing typos and grammatical mistakes.

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Code review formatting

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Add a link to the data consent section at the customize index (#9828)

* Add a link to the data consent section at the customize index

* Add link to customize logic page

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Fix preserving bold text in the rich text editor when pasting content (#9640)

* Fix editor clipboard pasting losting bold types

* Add a spec testing the rich text editor clipboard pasting

* Add the license notes for the Quill clipboard overrides

* Fix importing a page component without a body (#9863)

* Add page in AssemblyImporter JSON example

* Add spec for pages imported without body

* Add guard clause for pages imported without body

* Fix duplicate stats on home page and participatory space main page (#9624)

* Fix home page and participatory space duplicate stats

* Test that there are no duplicate stats with the presenters

* Remove `axe-core-rspec` dependency to get fully rid of `virtus` dependency (#9758)

* Remove axe-core-rspec dependency and add a custom matcher instead

In order to get rid of the `virtus` dependency, remove
axe-core-rspec and replace it with a custom matcher implementing
essentially the same functionality.

* Rake bundle

* Load the axe JS library always to make it available for the view

* Do not print indented blank lines for blank messages

* Add a spec for the axe matchers

* Add 2 levels of indentation to the node messages

To better match the output with the old format.

* Fix the expected message format in the spec after changing it

* Format the code slightly better

* Replace social-share-button gem with social share services API (#9545)

* Remove social-share-button gem

* fixup! Remove social-share-button gem

* Add SocialShareServiceManifest

* Add social share services registry with the social networks

* Add configuration in generated apps (initializer and secrets)

* Add social share button helper

* Add documentation

* Add CHANGELOG entry for existing apps

* s/Share on/Share to/g

* Extract render_social_share_button to its own method

* Escape URLs when sharing

* Add optional params support

* Fix typo

* Add spec for  render_social_share_button (Email and Twitter)

* Refactor regexp

* Remove unsupported logos for social share services

* Add documentation

* Fix method signature after Ruby 3 upgrade

* Fix rubocop offense

* Remove social-share-button gem javascript

* Remove unused social services from SCSS

* Remove unecessary use of the double-splat operator

* Remove social-share-button gem javascript from the redesign pack

* Fix issue with optional parameter value provided as nil

* Resque from KeyError when calling the format method in the manifest

* Do not render the social share URL if required parameters are not provided

* Update the return statement documentation correctly

* Add alt attribute to the social share images

* Add the meta tags to the dummy resources show view

In order to make the meta URL available and to show the social
share buttons on the page.

* Do not compact the arguments in the excape method

To avoid any side-effects, compact the arguments already when
passing them to the method.

* Add line break at the end of the the CHANGELOG section

* Fix the WhatsApp related specs in the social share button specs

- Target the WhatsApp element correctly according to the new API
- Remove the different cases for mobile and desktop because the
  new API shows the same button for both devices

* Unify naming for the social media services registry and registering

Use the same convention for the social media services registry as
with other manifest registries.

* Rubocop

* Add the `DECIDIM_SOCIAL_SHARE_SERVICES` config to the env vars docs

* Proofreading social shares docs

* Markdownlint

* Fix merge issue with CHANGELOG and add relevant section number

* Run rake bundle

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>

* hide progress bar on subcategories

* item-list metadata mobile

* remove deprecated

* no changes for design-app in accountability

* remove category description

* remove deprecated file

* fix a11y issues

* Display component name in header instead of generic accountability name

* Fix path of first element of breadcrumb

* upgrade heading tag

* Return blank in linked_resources_for cell if no linked_resources found

* Hide tabs on project cell when associated panel is blank

* Add status to results card_l

* Fix metadata item_list styles

* new adjustment list metadata

* fix icon method

* Replace -> separator with -

* set color progress indicator

* Change scopes default icon and use resource_type_icon_key for categories and scopes

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>
Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com>
Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
entantoencuanto added a commit that referenced this pull request Nov 10, 2022
* develop:
  Fix running DB commands consecutively (#9607)
  Always allow image upload in WYSWYG editor (#10011)
  Fix usages of sanitize helper methods for editable content provided by admins (#10007)
  Show only ToS acceptance when admin hasn't accepted it (#9907)
  Redesign: accountability (#9772)
  Update the gitignore statement for development app (#10045)
  Improve RSpec configuration  (#9931)
  Refactor the meeting list item title display (#10043)
Quentinchampenois pushed a commit to Quentinchampenois/decidim that referenced this pull request Nov 23, 2022
* home accountability desktop

* extract filters to component

* index accountability desktop

* add headers & updated sidebar widget

* refactor status, mark as deprecated

* extract css components

* generalize drawer

* use all the core assets from app-design

* larger scope for stylelint overrides in gem stylesheets (not only core)

* missing semicolon stylelint

* home acc. mobile design

* filters redesigned for mobile (also profile)

* update dropdown/filters component

* update filters in user_activity

* enhance auto-close for dropdowns

* fix erblint

* move drawer to normal app

* refactor list of projects

* move generic metadata to css component

* remove deprecated

* cancel out padding right on reverse layouts

* Fix global moderation types not translated (decidim#9662)

* Fix unpublished processes shown in the group process count (decidim#9682)

* Fix a typo in the logic customization manual (decidim#9831)

* Extract status to a cell

* Move filter items calculation to a helper method

* Fix test to not use removed stylesheets imports

* Update selector in test

* Fix tags selectors in tests

* Fix linter offenses

* Remove unused translations

* Normalize translations file

* add inner-button default styles

* remove CSS duplicated as of 695e68a

* extract css filters

* Define a cell to display results data

* Define a category item in result cell

* Define a dates_item_compact element in result cell

* Add a variant of project cell with different template and items

* Allow status cell to accept models different to categories

* Fix condition to build items array

* Extend the use of status cell

* Define linked_resources_for at cell level

* Refactor results and projects views to use cells

* Remove unused partials

* Remove unecessary line in push notifications spec (decidim#9833)

* Fix updating organization settings in case there were errors (decidim#9634)

* Fix source format in the data consent docs for the example (decidim#9837)

* Redesign: conversations (decidim#9469)

* conversations index

* conversation show

* reformat scroll to bottom no jquery

* fix naming

* allow only one initialization on load

* scroll to last child fix on reloads

* conversations shows + first approach form validation no-abide

* Recover form-error class in error messages in forms

* Fix translations

* Call author cell with context actions

* Replace TODOs with REDESIGN_PENDING

* Fix rubocop offense

* Refactor conversations to allow creation

* Recover jQuery use

* Update test after abide removal

* Fix tests

* Allow set novalidate in decidim_form_for helper method

* Recover novalidate to true by default in decidim_form_for

* Move messages to controller and remove duplicated @Conversations

* Validate initiative fill_data form in front

* Fix typo and add validation_messages as helper_method

* extract conversation styles

* update markup

* generalization form error

* new redesign conversations

* new index conversations

* generalization character counter

* input moji feature

* fix messages layout

* fix wrapper char counter

* fix style issues

* update conversations/new mockup

* responsive aside adaptations

* aside buttons conversations

* reorder time in conversations

* unread condition

* ignore stylelint

* mobile conversations

* position emoji

* fix position helper messages on generic forms

* add character counter fix

* Recover abide in forms

* Fix reply js replacement after creating a conversation

* Remove unused translations

* Fix recovering of abide in forms

* Fix selectors in tests

* Recover old modal behavior

* Update expected texts in tests

* Hide test that may no longer be needed

* Update selector in test

* Update selectors in test

* Comment maybe deprecated example

* Remove redesign pending comments

* remove redupe dep

* Force id of new conversation modal form

* Replace main with div in wrapper layout part

This avoids accessibility errors testing redesigned parts using old
layouts

* tune aside layout for allow different setups in mobile

* Use an identifier for main section to be used in tests

* Update selector in tests looking for main

* Recover main element in legacy wrapper

* add css classes to shrink icon size

* fix desktop sidebar layout

* Extract display_name method in author cell

* Hide messages button in profile actions when user has muted conversations

* Display unread pending indicator in redesigned layout main links when there are unread conversations also

* Enable full redesign feature in conversations integration tests

* Fix tests of pending conversations

* Update tests with redesigned author cell

* Update tests of messages in profile

* Skip temporarily accessibility tests in conversations due to pending redesign issues

* Skip test pending of another PR merge to pass

* Recover skipped test

* Fix footer h sequence

* Fix not allowed “no-external-link” on element “a” accessibility error

* Replace class to hide dataconsent-warning element

* Replace time with invalid value format with div

* Replace invalid div inside summary with span

* Add alt to image tag

* Fix h sequence

* Avoid having too much elements inside label tag

* Remove repeated line

* Recover accessibility tests in conversations

* Recover move hidden class to attribute in dataconsent-warning element and fix javascript

* Recover hide class for compatibility with old design

* Change tag in two cols layout to ensure accessibility with old design

* Add extra check

* Define set_redesign using environment

* Attempt to enable redesign in tests

* Attempt to enable redesign in tests using configuration persisted in database

* Change primary color to improve contrast

* Force new commit

* Force redesign_active

* Redefine primary color

* Disable tweak_bootsnap

* Allow forcing redesign in controllers

* Adapt test to parts with redesign not enabled

* Revert "Force redesign_active"

This reverts commit 556172c.

* modal required files

* conversations modal and redesigned_icon jquery-free 🎉

* Replace div tag with span to ensure accesibility

* Replace div tag with span to ensure accesibility

* Fix linter offense

* Remove unused translation

* Revert "Disable tweak_bootsnap"

This reverts commit b23a400.

* Prepare follow feature related tests to both redesigned and not redesigned versions

* missing omniauth buttons

* fix initializer multiple-mentions

* fix autocomplete styles

* add disabled for buttons

* Add missing id

* Disable automatic id

* Remove deprecated tests

* Skip test using old modal inside a page that will be removed

* Recover hide class in condition for compatibility with old design

* Remove commented code

* Fix announce threshold calculation bug

* Move reply form to a partial and reuse it

* add i18n button title

* remove autofocus

* alter order buttons + shrink size

* move styles to file

* Fix translation alphabetical order

* reduce size button modal & align center sidebar buttons

* order aside actions

* add border transparent to default button

Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com>

* Fix i18n-tasks issues

* move filter search to css component

* restore search element

* Define a new kind of card to use for component resources in lists and create one for accountability results

* Update test

* Add metadata cell, use it in card_l and refactor some parts

* Refactor results card_l using metadata

* Update styles

* Recover drawer layout

* Update categories hierarchy display in filtered projects indexes

* Update the CHANGELOG template and fix detailed changes heading levels (decidim#9843)

* Update the changelog template and fix detailed changes heading levels

* Make the same updates to the current CHANGELOG

* Fix the heading level for the API change sub sections

* Add tailwind as an API change

* Fix typo

* Fix moderations for comments that are mapped to deleted resources (decidim#9621)

* Return nil for reported content url if the commentable is deleted

* Show "deleted resource" instead of the visit URL when URL is not returned

* Add system specs testing that moderations for comments with deleted resources work

* Fix issue hiding comments for deleted debates

* Add moderation test to hide comments for deleted resources

* Add manage moderations shared examples for debates

* Fix specs after merging with develop

* Do not import resources multiple times (decidim#9644)

* Fix form answer attachments breaking the answer view (decidim#9653)

* Improve release process (decidim#9492)

* Improve the release related comands

- Abort the process if there are uncommitted changes
- Abort the process in case the official locales are not complete
  or clean
- Do not release NPM packages in case some of the gem releases
  fail
- Add a feature to retry the commands and make it configurable
- Enable retrying the commands by default assuming using OTP
  authentication

* Disable the GemManager retries for the generators specs

* Configure retry times env variable not to retry infinitely

* Configure the RETRY_TIMES env var for the gem installations in spec

* Move the RETRY_TIMES configuration to the spec helper

* Rubocop

* Fix user related absolute URLs (decidim#9596)

* Fix the user related URLs to contain the correct port

* Add a spec for the profile_url method in UserPresenter

* Fix default_avatar_url in UserPresenter

* Fix specs broken by the port change

* Fix broken specs

* Call the same method in both cases

* Fix "disappearing" underscores with comments (decidim#9654)

The underscores were actually converted into `<em>`s which is not
desired.

* Fix editor content saving when the content has only one video (decidim#9629)

* Allow only videos on the rich text editor

* Add a spec testing adding only a video through Quill

* Fix date/time formats at component forms (decidim#9632)

* Fix the localized date and time with zone with ISO formats

* Fix the datetime_field with Time and DateTime

* Change the component time to TimeWithZone attribute type

* Fix the conferences admin update controller spec

* Fix last activity page showing recently updated records (decidim#9844)

* Fix last activity page showing recently updated records

* Fix missing verb

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Fix collaborative drafts page when there are errors on the form (decidim#9656)

* Fix the finite value on the debate form when editing an existing debate (decidim#9645)

* Fix importing participatory process from legacy format (decidim#9646)

* Fix exporting and importing a page component (decidim#9638)

* Fix webpacker crashes on missing icons (decidim#9636)

* Remove GIF and BMP from the default file formats (decidim#9641)

* Remove gif and bmp from the default allowed file formats

* Remove gif from the core examples

* Add results metadata in show footer

* Fix argument in meetings panel

* Hide progress when not present in status cell

* Allow displaying status on results breadcrumb even if there are no results

* Display attachments in result show (resesign pending)

* Hide scope filter when component has no subscopes

* hide global status from home if progress disabled in component

* Update selector to work in redesigned parts

* Skip test pending of redesign

* Change selectors in tests

* Remove not expected result in test

* Add interaction to tests

* Update expected content in test

* Skip comments tests pending of redesign

* Fix selector in test

* add CSS component drawer_metadata

* Remove unused argument of helper method

* Add missing semicolon in scss file

* Fix linter offense

* Add documentation page for accessibility tips (decidim#9608)

* Add page for accessibility tips

* Add the example for accessible icon links

* Typo fixes and refinements

* Refer to the accessibility guide from the styles customization

* Document common accessibility mistakes

* Valid example HTML

* Generalize list styling

* Add tips about headings

* Fix typo

* Update the page with the points from decidim#6207

* Apply suggestions from code review

Fixing typos and grammatical mistakes.

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Code review formatting

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Add a link to the data consent section at the customize index (decidim#9828)

* Add a link to the data consent section at the customize index

* Add link to customize logic page

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Fix preserving bold text in the rich text editor when pasting content (decidim#9640)

* Fix editor clipboard pasting losting bold types

* Add a spec testing the rich text editor clipboard pasting

* Add the license notes for the Quill clipboard overrides

* Fix importing a page component without a body (decidim#9863)

* Add page in AssemblyImporter JSON example

* Add spec for pages imported without body

* Add guard clause for pages imported without body

* Fix duplicate stats on home page and participatory space main page (decidim#9624)

* Fix home page and participatory space duplicate stats

* Test that there are no duplicate stats with the presenters

* Remove `axe-core-rspec` dependency to get fully rid of `virtus` dependency (decidim#9758)

* Remove axe-core-rspec dependency and add a custom matcher instead

In order to get rid of the `virtus` dependency, remove
axe-core-rspec and replace it with a custom matcher implementing
essentially the same functionality.

* Rake bundle

* Load the axe JS library always to make it available for the view

* Do not print indented blank lines for blank messages

* Add a spec for the axe matchers

* Add 2 levels of indentation to the node messages

To better match the output with the old format.

* Fix the expected message format in the spec after changing it

* Format the code slightly better

* Replace social-share-button gem with social share services API (decidim#9545)

* Remove social-share-button gem

* fixup! Remove social-share-button gem

* Add SocialShareServiceManifest

* Add social share services registry with the social networks

* Add configuration in generated apps (initializer and secrets)

* Add social share button helper

* Add documentation

* Add CHANGELOG entry for existing apps

* s/Share on/Share to/g

* Extract render_social_share_button to its own method

* Escape URLs when sharing

* Add optional params support

* Fix typo

* Add spec for  render_social_share_button (Email and Twitter)

* Refactor regexp

* Remove unsupported logos for social share services

* Add documentation

* Fix method signature after Ruby 3 upgrade

* Fix rubocop offense

* Remove social-share-button gem javascript

* Remove unused social services from SCSS

* Remove unecessary use of the double-splat operator

* Remove social-share-button gem javascript from the redesign pack

* Fix issue with optional parameter value provided as nil

* Resque from KeyError when calling the format method in the manifest

* Do not render the social share URL if required parameters are not provided

* Update the return statement documentation correctly

* Add alt attribute to the social share images

* Add the meta tags to the dummy resources show view

In order to make the meta URL available and to show the social
share buttons on the page.

* Do not compact the arguments in the excape method

To avoid any side-effects, compact the arguments already when
passing them to the method.

* Add line break at the end of the the CHANGELOG section

* Fix the WhatsApp related specs in the social share button specs

- Target the WhatsApp element correctly according to the new API
- Remove the different cases for mobile and desktop because the
  new API shows the same button for both devices

* Unify naming for the social media services registry and registering

Use the same convention for the social media services registry as
with other manifest registries.

* Rubocop

* Add the `DECIDIM_SOCIAL_SHARE_SERVICES` config to the env vars docs

* Proofreading social shares docs

* Markdownlint

* Fix merge issue with CHANGELOG and add relevant section number

* Run rake bundle

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>

* hide progress bar on subcategories

* item-list metadata mobile

* remove deprecated

* no changes for design-app in accountability

* remove category description

* remove deprecated file

* fix a11y issues

* Display component name in header instead of generic accountability name

* Fix path of first element of breadcrumb

* upgrade heading tag

* Return blank in linked_resources_for cell if no linked_resources found

* Hide tabs on project cell when associated panel is blank

* Add status to results card_l

* Fix metadata item_list styles

* new adjustment list metadata

* fix icon method

* Replace -> separator with -

* set color progress indicator

* Change scopes default icon and use resource_type_icon_key for categories and scopes

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>
Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com>
Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
ahukkanen added a commit that referenced this pull request Dec 16, 2022
* create new cells

* replace hard-coded blog buttons with cells

* remove externalized component from blogs

* extract css component & general fixes for cells

* endorsers list toggle

* Refactor endorsers_list cell to inherit from RedesignedButtonCell

* Refactor comments_button cell to inherit from RedesignedButtonCell

* Refactor share and flag buttons to inherit from RedesignedButtonCell

* Simplify redesigned button cell

* share modal redesign

* Remove unused translations

* Fix linter offense

* Split social share button and related tests in redesigned and not redesigned versions

* provide social network icon colors

* report button triggers flag modal

* don't use an anchor if no href

* identity modal

* endorsement auth modals

* Merge branch 'develop' into feature/redesign-action-buttons

* mark old script as deprecated in jsdoc

* allow remote, jquery-free modals 🎉

* Fix modal

* Disable dropdown behavior if no endorsements present in endorsers list

* Move remote modal initialization to redesigned_index

* Include Decidim::FiltersHelper at application controller level

* Split authorization modal in redesigned and legacy versions

* override dropdown defaults in endorsers-list

* Split authorization modal in redesigned and legacy versions

* Move parts of redesigned authorizations modal content to controller

* Sanitize translations

* Fix linter offense

* Fix tests providing redesign_enabled? method

* Update tests and social_share_button_tag method

* Add parameters to Dialog initializer optionally to avoid accesibility errors

* Avoid blank elements in verifications of authorization modal

* Ensure main tag presence in layout_center

* Fix usages of `reorder` and `paginate` (#9926)

* Fix organization SMTP password not saved (became blank) in system panel (#9925)

* Fix organization SMTP password not saved (became blank) in system panel

* Fix organization SMTP password not saved (became blank) in system panel - refactoring

* Reformat CSV help for import files on Accountability and Proposals (#9891)

* Move explanation in accountability's admin import CSV results

We also remove the 'import-process-info' class and its CSS definition,
as it isn't necessary anymore.

* Add HTML support to ImportForm help text

* Reformat help text to proposals' answers import

* Reformat help text to proposals' import

* Remove links with link word

* Add nested list in explanation

* Add an example for English language on i18n attributes

* Refactor cell titles (#10032)

* Refactor HTML escaping in cell titles

* Add specs to test the cell titles

* Fix meetings spec

* Split CHANGELOG and RELEASE_NOTES (#10031)

* Split CHANGELOG and RELEASE_NOTES to their own documents

* Fix the heading levels in the changelog generator

* Update releases documentation about CHANGELOG and RELEASE_NOTES

* Rename to "changelog" instead of "change log"

* Update the RELEASE_NOTES structure to make it easier to copy-paste

* Update the RELEASE_NOTES instructions in releases documentation

* Fix the previous versions link in CHANGELOG

* Fix the release title

* Refactor admin listing titles (#10042)

* Refactor the meeting list item title display (#10043)

* Refactor the meeting list item title display

* Remove unnecessary method

* Refactor meeting list item

* Update the meeting list item spec

* Add spec for the highlighted list item cell

* Improve RSpec configuration  (#9931)

* Improve RSpec configuration

* Bump version for Antora

* Update the gitignore statement for development app (#10045)

* Redesign: accountability (#9772)

* home accountability desktop

* extract filters to component

* index accountability desktop

* add headers & updated sidebar widget

* refactor status, mark as deprecated

* extract css components

* generalize drawer

* use all the core assets from app-design

* larger scope for stylelint overrides in gem stylesheets (not only core)

* missing semicolon stylelint

* home acc. mobile design

* filters redesigned for mobile (also profile)

* update dropdown/filters component

* update filters in user_activity

* enhance auto-close for dropdowns

* fix erblint

* move drawer to normal app

* refactor list of projects

* move generic metadata to css component

* remove deprecated

* cancel out padding right on reverse layouts

* Fix global moderation types not translated (#9662)

* Fix unpublished processes shown in the group process count (#9682)

* Fix a typo in the logic customization manual (#9831)

* Extract status to a cell

* Move filter items calculation to a helper method

* Fix test to not use removed stylesheets imports

* Update selector in test

* Fix tags selectors in tests

* Fix linter offenses

* Remove unused translations

* Normalize translations file

* add inner-button default styles

* remove CSS duplicated as of 695e68a

* extract css filters

* Define a cell to display results data

* Define a category item in result cell

* Define a dates_item_compact element in result cell

* Add a variant of project cell with different template and items

* Allow status cell to accept models different to categories

* Fix condition to build items array

* Extend the use of status cell

* Define linked_resources_for at cell level

* Refactor results and projects views to use cells

* Remove unused partials

* Remove unecessary line in push notifications spec (#9833)

* Fix updating organization settings in case there were errors (#9634)

* Fix source format in the data consent docs for the example (#9837)

* Redesign: conversations (#9469)

* conversations index

* conversation show

* reformat scroll to bottom no jquery

* fix naming

* allow only one initialization on load

* scroll to last child fix on reloads

* conversations shows + first approach form validation no-abide

* Recover form-error class in error messages in forms

* Fix translations

* Call author cell with context actions

* Replace TODOs with REDESIGN_PENDING

* Fix rubocop offense

* Refactor conversations to allow creation

* Recover jQuery use

* Update test after abide removal

* Fix tests

* Allow set novalidate in decidim_form_for helper method

* Recover novalidate to true by default in decidim_form_for

* Move messages to controller and remove duplicated @Conversations

* Validate initiative fill_data form in front

* Fix typo and add validation_messages as helper_method

* extract conversation styles

* update markup

* generalization form error

* new redesign conversations

* new index conversations

* generalization character counter

* input moji feature

* fix messages layout

* fix wrapper char counter

* fix style issues

* update conversations/new mockup

* responsive aside adaptations

* aside buttons conversations

* reorder time in conversations

* unread condition

* ignore stylelint

* mobile conversations

* position emoji

* fix position helper messages on generic forms

* add character counter fix

* Recover abide in forms

* Fix reply js replacement after creating a conversation

* Remove unused translations

* Fix recovering of abide in forms

* Fix selectors in tests

* Recover old modal behavior

* Update expected texts in tests

* Hide test that may no longer be needed

* Update selector in test

* Update selectors in test

* Comment maybe deprecated example

* Remove redesign pending comments

* remove redupe dep

* Force id of new conversation modal form

* Replace main with div in wrapper layout part

This avoids accessibility errors testing redesigned parts using old
layouts

* tune aside layout for allow different setups in mobile

* Use an identifier for main section to be used in tests

* Update selector in tests looking for main

* Recover main element in legacy wrapper

* add css classes to shrink icon size

* fix desktop sidebar layout

* Extract display_name method in author cell

* Hide messages button in profile actions when user has muted conversations

* Display unread pending indicator in redesigned layout main links when there are unread conversations also

* Enable full redesign feature in conversations integration tests

* Fix tests of pending conversations

* Update tests with redesigned author cell

* Update tests of messages in profile

* Skip temporarily accessibility tests in conversations due to pending redesign issues

* Skip test pending of another PR merge to pass

* Recover skipped test

* Fix footer h sequence

* Fix not allowed “no-external-link” on element “a” accessibility error

* Replace class to hide dataconsent-warning element

* Replace time with invalid value format with div

* Replace invalid div inside summary with span

* Add alt to image tag

* Fix h sequence

* Avoid having too much elements inside label tag

* Remove repeated line

* Recover accessibility tests in conversations

* Recover move hidden class to attribute in dataconsent-warning element and fix javascript

* Recover hide class for compatibility with old design

* Change tag in two cols layout to ensure accessibility with old design

* Add extra check

* Define set_redesign using environment

* Attempt to enable redesign in tests

* Attempt to enable redesign in tests using configuration persisted in database

* Change primary color to improve contrast

* Force new commit

* Force redesign_active

* Redefine primary color

* Disable tweak_bootsnap

* Allow forcing redesign in controllers

* Adapt test to parts with redesign not enabled

* Revert "Force redesign_active"

This reverts commit 556172c.

* modal required files

* conversations modal and redesigned_icon jquery-free 🎉

* Replace div tag with span to ensure accesibility

* Replace div tag with span to ensure accesibility

* Fix linter offense

* Remove unused translation

* Revert "Disable tweak_bootsnap"

This reverts commit b23a400.

* Prepare follow feature related tests to both redesigned and not redesigned versions

* missing omniauth buttons

* fix initializer multiple-mentions

* fix autocomplete styles

* add disabled for buttons

* Add missing id

* Disable automatic id

* Remove deprecated tests

* Skip test using old modal inside a page that will be removed

* Recover hide class in condition for compatibility with old design

* Remove commented code

* Fix announce threshold calculation bug

* Move reply form to a partial and reuse it

* add i18n button title

* remove autofocus

* alter order buttons + shrink size

* move styles to file

* Fix translation alphabetical order

* reduce size button modal & align center sidebar buttons

* order aside actions

* add border transparent to default button

Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com>

* Fix i18n-tasks issues

* move filter search to css component

* restore search element

* Define a new kind of card to use for component resources in lists and create one for accountability results

* Update test

* Add metadata cell, use it in card_l and refactor some parts

* Refactor results card_l using metadata

* Update styles

* Recover drawer layout

* Update categories hierarchy display in filtered projects indexes

* Update the CHANGELOG template and fix detailed changes heading levels (#9843)

* Update the changelog template and fix detailed changes heading levels

* Make the same updates to the current CHANGELOG

* Fix the heading level for the API change sub sections

* Add tailwind as an API change

* Fix typo

* Fix moderations for comments that are mapped to deleted resources (#9621)

* Return nil for reported content url if the commentable is deleted

* Show "deleted resource" instead of the visit URL when URL is not returned

* Add system specs testing that moderations for comments with deleted resources work

* Fix issue hiding comments for deleted debates

* Add moderation test to hide comments for deleted resources

* Add manage moderations shared examples for debates

* Fix specs after merging with develop

* Do not import resources multiple times (#9644)

* Fix form answer attachments breaking the answer view (#9653)

* Improve release process (#9492)

* Improve the release related comands

- Abort the process if there are uncommitted changes
- Abort the process in case the official locales are not complete
  or clean
- Do not release NPM packages in case some of the gem releases
  fail
- Add a feature to retry the commands and make it configurable
- Enable retrying the commands by default assuming using OTP
  authentication

* Disable the GemManager retries for the generators specs

* Configure retry times env variable not to retry infinitely

* Configure the RETRY_TIMES env var for the gem installations in spec

* Move the RETRY_TIMES configuration to the spec helper

* Rubocop

* Fix user related absolute URLs (#9596)

* Fix the user related URLs to contain the correct port

* Add a spec for the profile_url method in UserPresenter

* Fix default_avatar_url in UserPresenter

* Fix specs broken by the port change

* Fix broken specs

* Call the same method in both cases

* Fix "disappearing" underscores with comments (#9654)

The underscores were actually converted into `<em>`s which is not
desired.

* Fix editor content saving when the content has only one video (#9629)

* Allow only videos on the rich text editor

* Add a spec testing adding only a video through Quill

* Fix date/time formats at component forms (#9632)

* Fix the localized date and time with zone with ISO formats

* Fix the datetime_field with Time and DateTime

* Change the component time to TimeWithZone attribute type

* Fix the conferences admin update controller spec

* Fix last activity page showing recently updated records (#9844)

* Fix last activity page showing recently updated records

* Fix missing verb

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Fix collaborative drafts page when there are errors on the form (#9656)

* Fix the finite value on the debate form when editing an existing debate (#9645)

* Fix importing participatory process from legacy format (#9646)

* Fix exporting and importing a page component (#9638)

* Fix webpacker crashes on missing icons (#9636)

* Remove GIF and BMP from the default file formats (#9641)

* Remove gif and bmp from the default allowed file formats

* Remove gif from the core examples

* Add results metadata in show footer

* Fix argument in meetings panel

* Hide progress when not present in status cell

* Allow displaying status on results breadcrumb even if there are no results

* Display attachments in result show (resesign pending)

* Hide scope filter when component has no subscopes

* hide global status from home if progress disabled in component

* Update selector to work in redesigned parts

* Skip test pending of redesign

* Change selectors in tests

* Remove not expected result in test

* Add interaction to tests

* Update expected content in test

* Skip comments tests pending of redesign

* Fix selector in test

* add CSS component drawer_metadata

* Remove unused argument of helper method

* Add missing semicolon in scss file

* Fix linter offense

* Add documentation page for accessibility tips (#9608)

* Add page for accessibility tips

* Add the example for accessible icon links

* Typo fixes and refinements

* Refer to the accessibility guide from the styles customization

* Document common accessibility mistakes

* Valid example HTML

* Generalize list styling

* Add tips about headings

* Fix typo

* Update the page with the points from #6207

* Apply suggestions from code review

Fixing typos and grammatical mistakes.

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Code review formatting

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Add a link to the data consent section at the customize index (#9828)

* Add a link to the data consent section at the customize index

* Add link to customize logic page

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Fix preserving bold text in the rich text editor when pasting content (#9640)

* Fix editor clipboard pasting losting bold types

* Add a spec testing the rich text editor clipboard pasting

* Add the license notes for the Quill clipboard overrides

* Fix importing a page component without a body (#9863)

* Add page in AssemblyImporter JSON example

* Add spec for pages imported without body

* Add guard clause for pages imported without body

* Fix duplicate stats on home page and participatory space main page (#9624)

* Fix home page and participatory space duplicate stats

* Test that there are no duplicate stats with the presenters

* Remove `axe-core-rspec` dependency to get fully rid of `virtus` dependency (#9758)

* Remove axe-core-rspec dependency and add a custom matcher instead

In order to get rid of the `virtus` dependency, remove
axe-core-rspec and replace it with a custom matcher implementing
essentially the same functionality.

* Rake bundle

* Load the axe JS library always to make it available for the view

* Do not print indented blank lines for blank messages

* Add a spec for the axe matchers

* Add 2 levels of indentation to the node messages

To better match the output with the old format.

* Fix the expected message format in the spec after changing it

* Format the code slightly better

* Replace social-share-button gem with social share services API (#9545)

* Remove social-share-button gem

* fixup! Remove social-share-button gem

* Add SocialShareServiceManifest

* Add social share services registry with the social networks

* Add configuration in generated apps (initializer and secrets)

* Add social share button helper

* Add documentation

* Add CHANGELOG entry for existing apps

* s/Share on/Share to/g

* Extract render_social_share_button to its own method

* Escape URLs when sharing

* Add optional params support

* Fix typo

* Add spec for  render_social_share_button (Email and Twitter)

* Refactor regexp

* Remove unsupported logos for social share services

* Add documentation

* Fix method signature after Ruby 3 upgrade

* Fix rubocop offense

* Remove social-share-button gem javascript

* Remove unused social services from SCSS

* Remove unecessary use of the double-splat operator

* Remove social-share-button gem javascript from the redesign pack

* Fix issue with optional parameter value provided as nil

* Resque from KeyError when calling the format method in the manifest

* Do not render the social share URL if required parameters are not provided

* Update the return statement documentation correctly

* Add alt attribute to the social share images

* Add the meta tags to the dummy resources show view

In order to make the meta URL available and to show the social
share buttons on the page.

* Do not compact the arguments in the excape method

To avoid any side-effects, compact the arguments already when
passing them to the method.

* Add line break at the end of the the CHANGELOG section

* Fix the WhatsApp related specs in the social share button specs

- Target the WhatsApp element correctly according to the new API
- Remove the different cases for mobile and desktop because the
  new API shows the same button for both devices

* Unify naming for the social media services registry and registering

Use the same convention for the social media services registry as
with other manifest registries.

* Rubocop

* Add the `DECIDIM_SOCIAL_SHARE_SERVICES` config to the env vars docs

* Proofreading social shares docs

* Markdownlint

* Fix merge issue with CHANGELOG and add relevant section number

* Run rake bundle

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>

* hide progress bar on subcategories

* item-list metadata mobile

* remove deprecated

* no changes for design-app in accountability

* remove category description

* remove deprecated file

* fix a11y issues

* Display component name in header instead of generic accountability name

* Fix path of first element of breadcrumb

* upgrade heading tag

* Return blank in linked_resources_for cell if no linked_resources found

* Hide tabs on project cell when associated panel is blank

* Add status to results card_l

* Fix metadata item_list styles

* new adjustment list metadata

* fix icon method

* Replace -> separator with -

* set color progress indicator

* Change scopes default icon and use resource_type_icon_key for categories and scopes

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>
Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com>
Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* Show only ToS acceptance when admin hasn't accepted it (#9907)

* Don't show the 'Activity' card in dashboard if admin didn't accepted the ToS

* Don't show the 'Global moderations' menu if admin didn't accepted the ToS

* Extract spec to a shared example

* Don't show the 'Participants' menu if admin didn't accepted the ToS

* Fix CTA button copy

* Don't show the 'Templates' menu if admin didn't accepted the ToS

* Don't show the 'Initiatives' menu if admin didn't accepted the ToS

* Add system spec

* Fix specs

* Add example for ToS acceptance in 'Global moderations'

* Fix spec

* Add example for ToS acceptance in 'Space moderations'

* Fix guard clause when admin is managing users

* Fix usages of sanitize helper methods for editable content provided by admins (#10007)

* Fix usages of sanitize helper methods for editable content provided by admins.

Related issue: #9927

Fixes usages of sanitize for `description` of `ParticipatoryProcess` and `ParticipatoryProcessGroup`.

* Fix usages of sanitize for `description` of assembly, conference and voting. Unify spec test.

* Fix usages of sanitize for `short_description` of conference.

* Always allow image upload in WYSWYG editor (#10011)

* Disable editor_images option in WYSWYG editor

* Correct rubocop offenses

* shared html styles for modals

* move action button to the right if it's the only child

* this block removal is within 38ac609, but it would contains conflicts to cherrypick it

* modal resposive refactor

* update modals markup

* fix share modal mobile

* refactor conversations modal

* fix stylelint

* change social share manifest attribute

* Call renamed attribute correctly

* limit the scope of th einitializer js function

* limit scope of frame-render initializer

* Remove redesigned_flag_modal from layout

* Move flag modal to report_button cell

* Make blog posts reportable

* fix erblint

* Add initial value to some functions and fix references to document

* standarize jsdoc

* fix social icon name

* Fix partial to include class in div conditionally

* add documentation for social share setup

* close modals on click outside

* keep modal focus

* Recover old options in redesign_authorized_to

* Use button instead of link in redesigned_follow_button

* Remove not required translation

* Replace endorse with like in button

* Remove report button from blog posts

* Revert reportable concern inclusion in blog posts

This reverts partially commit eb76be2.

* Make cache_hash private in report button

* add copy text to button

* add id to trigger button endorsement cell

* Move authorizations modal to a cell

* handle external link warning modal

* Fix expected button texts on endosement tests

* Use link processed by controller instead of using param directly

* Remove unused translation

* Disable temporarily test after redesign

* replace modal warning if already exists

* don't use rails helper on simple view

* remove deprecated helper

* Clarify comment

* hide button if no endorsements

* button translations

* show the plus indicator conditionally

* toggle icons like/dislike

* move icon keys to icon_helper

* fix debates test

* Update decidim-blogs/config/locales/en.yml

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

* replace identity list with a11y code

* remove no necessary

* Define a cell to wrap the block of endorsers and endorse button

* Move content of endorse button to a partial inside cell

* Define action to refresh endorse button content and endorsers list

* Use default implementation for comments button

* Define function in js.erb only if redesign is enabled to avoid errors

* handle remote modals special case

* fix linter

* short line

* fix fix fix

* Fix linter offense

* iterate over children with no variable mutation

* Don't display dialog modal trigger if remote_url blank

* add button to the remote loader

* clearer buttons

* Set nil as default for method in redesigned_button cell

* replace all available h3 dialog with h2

* add tabindex by default to titles

* set focus on title & better handling remote-modals

* Avoid initialize twice on turbo events

* dont show svg on hover/focus

* Change selector to identify like button

* Update whole like button if user doesn't have to select an identity

* Extract part of cell to template

* Display elements of endorsers button and list even when there are no endorsers

* wrap text into a span for truncate purposes

Co-authored-by: Eduardo Martinez Echevarria <eduardomech@gmail.com>
Co-authored-by: Sergei Krylov <sergei.krylov@yahoo.com>
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>
Co-authored-by: Sina Eftekhar <104360479+sinaeftekhar@users.noreply.github.com>
Co-authored-by: Fernando Blat <fernando@blat.es>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

project: redesign Barcelona City Council contract

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants