Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1712: Added “Expand all steps” link on step by step #354

Merged
merged 3 commits into from
Jul 8, 2024

Conversation

rimi-itk
Copy link

@rimi-itk rimi-itk commented Jun 26, 2024

https://leantime.itkdev.dk/dashboard/home#/tickets/showTicket/1712

Adds “Expand all steps” link to Step by step paragraphs.

Before:

Screen Shot 2024-06-26 at 16 55 17

After:

Screen Shot 2024-06-26 at 16 55 06

After clicking “Expand all steps”:

Screen Shot 2024-06-26 at 16 55 34

@rimi-itk rimi-itk force-pushed the feature/1712-step-by-step branch 2 times, most recently from 682c2e8 to 27f4fda Compare June 26, 2024 14:51
@rimi-itk rimi-itk marked this pull request as ready for review June 26, 2024 14:54
@rimi-itk rimi-itk requested a review from sinejespersen June 26, 2024 14:54
@rimi-itk rimi-itk force-pushed the feature/1712-step-by-step branch from 27f4fda to e734505 Compare June 26, 2024 14:58
<span class="bold">
{{ has_title ? content.os2loop_documents_step_title : content.os2loop_documents_step_text }}
</span>
</span>
{% else %}
<a class="collapsed step--collapse-toggle" type="button" data-toggle="collapse" data-target="#paragraph-id-{{ paragraph.id() }}">
<a class="{{ class_collapsed }} step--collapse-toggle" type="button" data-toggle="{{ class_collapse }}" data-target="#paragraph-id-{{ paragraph.id() }}">

Choose a reason for hiding this comment

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

why not button?

Copy link
Author

Choose a reason for hiding this comment

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

Because I didn't write the code. But it makes sense to change it.

Copy link
Author

Choose a reason for hiding this comment

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

The a element contains the step header (title) and a button element may not be appropriate for this use.

@sinejespersen sinejespersen self-requested a review June 27, 2024 07:58
@rimi-itk rimi-itk force-pushed the feature/1712-step-by-step branch from 69a5f43 to a01a0aa Compare July 5, 2024 12:04
@rimi-itk rimi-itk requested a review from sinejespersen July 5, 2024 13:24
@martinyde martinyde merged commit 1dd17ad into develop Jul 8, 2024
7 checks passed
@martinyde martinyde deleted the feature/1712-step-by-step branch July 8, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants