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

Fix formatting of bulleted list #1879

Merged
merged 2 commits into from
Aug 21, 2024

Conversation

sbabcoc
Copy link
Contributor

@sbabcoc sbabcoc commented Aug 21, 2024

User description

Description

Fix minor formatting issue

Motivation and Context

Making things pretty

Types of changes

  • Change to the site (I have double-checked the Netlify deployment, and my changes look good)
  • Code example added (and I also added the example to all translated languages)
  • Improved translation
  • Added new translation (and I also added a notice to each document missing translation)

Checklist

  • I have read the contributing document.
  • I have used hugo to render the site/docs locally and I am sure it works.

PR Type

documentation


Description

  • Fixed the formatting of a bulleted list in the documentation to enhance readability.
  • Removed unnecessary indentation from list items to ensure consistent styling.

Changes walkthrough 📝

Relevant files
Formatting
htmlunit-remote-for-selenium-4-grid.md
Fix formatting of bulleted list in documentation                 

website_and_docs/content/blog/2024/htmlunit-remote-for-selenium-4-grid.md

  • Fixed formatting of bulleted list.
  • Removed unnecessary indentation for list items.
  • +3/-3     

    💡 PR-Agent usage:
    Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    Copy link

    netlify bot commented Aug 21, 2024

    Deploy Preview for selenium-dev ready!

    Name Link
    🔨 Latest commit b04f7a9
    🔍 Latest deploy log https://app.netlify.com/sites/selenium-dev/deploys/66c59286fb16420008976af4
    😎 Deploy Preview https://deploy-preview-1879--selenium-dev.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify site configuration.

    @codiumai-pr-agent-pro codiumai-pr-agent-pro bot added documentation Improvements or additions to documentation Review effort [1-5]: 1 labels Aug 21, 2024
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No key issues to review

    Copy link
    Contributor

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Add a space after each asterisk in the bulleted list for better readability and consistent formatting

    Consider adding a space after each asterisk in the bulleted list to improve
    readability and ensure consistent formatting across different Markdown renderers.

    website_and_docs/content/blog/2024/htmlunit-remote-for-selenium-4-grid.md [42-44]

    -+* Create and manage driver sessions
    -+* Route driver commands to specified driver sessions
    -+* Package driver method results into HTTP responses
    ++ * Create and manage driver sessions
    ++ * Route driver commands to specified driver sessions
    ++ * Package driver method results into HTTP responses
     
    • Apply this suggestion
    Suggestion importance[1-10]: 7

    Why: The suggestion improves readability and ensures consistent formatting across different Markdown renderers by adding a space after each asterisk in the bulleted list. This is a minor enhancement but contributes to better presentation.

    7

    Copy link
    Member

    @diemol diemol left a comment

    Choose a reason for hiding this comment

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

    Thank you, @sbabcoc!

    @diemol diemol merged commit 50ae91e into SeleniumHQ:trunk Aug 21, 2024
    6 checks passed
    selenium-ci added a commit that referenced this pull request Aug 21, 2024
    Co-authored-by: Diego Molina <[email protected]>
    
    [deploy site] 50ae91e
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    documentation Improvements or additions to documentation Review effort [1-5]: 1
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants