Skip to content

Vika_progress_bar #1500

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Vika_progress_bar #1500

wants to merge 4 commits into from

Conversation

WildSeaTurtle
Copy link
Collaborator

No description provided.

Replaced outdated progress bar images with updated versions, including light and dark mode variants for improved consistency and clarity. Archived older assets into a dedicated "old" folder to maintain organization and support future updates.
Replaced outdated images with updated light and dark mode variants for better visual consistency. Restructured documentation to use tables for improved readability and clarity. Enhanced descriptions and examples throughout.
Replaced outdated progress bar images with updated light and dark mode variants for visual consistency. Added new assets for improved representation while archiving older ones. Simplified structure to better support future enhancements and organization.
Simplified content structure for improved readability. Consolidated determinate and indeterminate progress bar guidance with updated images. Enhanced descriptions for process details, control, and completion. Added new visuals and removed redundant examples for clarity.
Copy link
Collaborator

@eldar-jetbrains eldar-jetbrains left a comment

Choose a reason for hiding this comment

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

Left some comments here and in Figma

<img src="08 header_for_several_processes.png" alt="Header for several processes"/>
</td>
<td>
<p>If there are several processes in a group, add a bold header. The header is a noun.</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

should be

<p>Do not show a process name for inline processes.</p>
</td>
</tr>
</table>

#### Process details
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe add somewhere here what process detail can be and why to show it

Copy link
Collaborator

Choose a reason for hiding this comment

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

There is a link above, but it can be forgotten by the time you come to this section, so maybe mention it here as well

<p>If space is limited, show percentage completed.</p>
</td>
</tr>
</table>

#### In dialog
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe add a separate chapter 'Placement':
Placement:

  • In a dialog
  • In the status bar
  • In a tool window + show the progress bar in the tool window from Git

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