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

DEVPROD-5307: Documentation for Parsley Sectioning #8346

Merged
merged 5 commits into from
Oct 7, 2024

Conversation

SupaJoon
Copy link
Contributor

DEVPROD-5307

Description

These code changes introduce documentation for the Parsley sectioning feature.

@SupaJoon SupaJoon requested a review from a team September 30, 2024 15:49
docs/UI/Parsley-Sectioning.md Outdated Show resolved Hide resolved
docs/UI/Parsley-Sectioning.md Outdated Show resolved Hide resolved
Comment on lines 17 to 19
Clicking on the caret will also open and close a single command or function section.

![Toggle Single Section](../images/section_toggle.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you can omit this image (since it looks kinda odd just having the image of the caret) and just say

Suggested change
Clicking on the caret will also open and close a single command or function section.
![Toggle Single Section](../images/section_toggle.png)
The caret on the left-hand side can also be used to open or close a single command or function section.

which the reader can infer from the picture directly above

Copy link
Contributor

Choose a reason for hiding this comment

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

(can delete this file I think)

docs/UI/Parsley-Sectioning.md Outdated Show resolved Hide resolved
Sections that contain a search result will open. Iterating over search results will open the section if it's closed.

### Filtering
Sections will temporarily go away when filters are applied and come back when filters are removed.
Copy link
Contributor

Choose a reason for hiding this comment

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

(or something along these lines)

Suggested change
Sections will temporarily go away when filters are applied and come back when filters are removed.
Sectioning cannot be used in conjunction with filters. Sections will not render when filters are applied, but will reappear when all filters are removed.

Comment on lines +37 to +38
### Shareline
When a link with a share line is opened, Parsley will open the section that contains the share line and scroll to the line. The failing line section will still open but Parsley will scroll to the share line instead.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Shareline
When a link with a share line is opened, Parsley will open the section that contains the share line and scroll to the line. The failing line section will still open but Parsley will scroll to the share line instead.
### Share Line
Share line takes precedence over failing line. When a link with a share line is opened, Parsley will open the section that contains the share line and scroll to the line. The failing line section will still be open and accessible via the bookmarks bar.

## Overview
Parsley sectioning is toggleable from the "Log Viewing" tab of the "Parsley Settings" menu and is available to task logs only. Toggling the feature will render your task log into expandable sections split up by function and command.

## Feature details
Copy link
Contributor

Choose a reason for hiding this comment

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

nit. Would just recommend double checking if you want these headers in title case or in sentence case since it's slightly inconsistent (e.g. Section status vs Jump to Failing Line)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good one. I'll stick to sentence case.

SupaJoon and others added 4 commits October 7, 2024 12:22
Co-authored-by: minnakt <47064971+minnakt@users.noreply.github.com>
Co-authored-by: minnakt <47064971+minnakt@users.noreply.github.com>
Co-authored-by: minnakt <47064971+minnakt@users.noreply.github.com>
@SupaJoon SupaJoon merged commit de843f9 into evergreen-ci:main Oct 7, 2024
10 checks passed
@SupaJoon SupaJoon deleted the DEVPROD-5307 branch October 7, 2024 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants