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

Added ElementNotVisibleException to Error Documentation #1971

Open
wants to merge 10 commits into
base: trunk
Choose a base branch
from

Conversation

shbenzer
Copy link
Contributor

@shbenzer shbenzer commented Sep 24, 2024

User description

Added documentation coverage for ElementNotVisibleException

Description

added documentation for ElementNotVisibleException
updated all translations of common error's index.md

Motivation and Context

make documentation more comprehensive

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

  • Added documentation for ElementNotVisibleException across multiple languages (English, Japanese, Portuguese, Chinese).
  • Included sections on likely causes and possible solutions for the exception.

Changes walkthrough 📝

Relevant files
Documentation
_index.en.md
Document `ElementNotVisibleException` in English                 

website_and_docs/content/documentation/webdriver/troubleshooting/errors/_index.en.md

  • Added documentation for ElementNotVisibleException.
  • Described likely causes and possible solutions.
  • +16/-0   
    _index.ja.md
    Document `ElementNotVisibleException` in Japanese               

    website_and_docs/content/documentation/webdriver/troubleshooting/errors/_index.ja.md

  • Added documentation for ElementNotVisibleException.
  • Described likely causes and possible solutions.
  • +16/-0   
    _index.pt-br.md
    Document `ElementNotVisibleException` in Portuguese           

    website_and_docs/content/documentation/webdriver/troubleshooting/errors/_index.pt-br.md

  • Added documentation for ElementNotVisibleException.
  • Described likely causes and possible solutions.
  • +16/-0   
    _index.zh-cn.md
    Document `ElementNotVisibleException` in Chinese                 

    website_and_docs/content/documentation/webdriver/troubleshooting/errors/_index.zh-cn.md

  • Added documentation for ElementNotVisibleException.
  • Described likely causes and possible solutions.
  • +16/-0   

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    Copy link

    netlify bot commented Sep 24, 2024

    👷 Deploy request for selenium-dev pending review.

    Visit the deploys page to approve it

    Name Link
    🔨 Latest commit 45a1645

    @qodo-merge-pro qodo-merge-pro bot added the documentation Improvements or additions to documentation label Sep 24, 2024
    Copy link
    Contributor

    PR Reviewer Guide 🔍

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

    Untranslated Content
    The new content added to the Chinese version of the documentation is in English, not Chinese.

    Copy link
    Contributor

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Translate the newly added content to match the language of the file

    Translate the newly added content for ElementNotVisibleException into Chinese to
    maintain consistency with the rest of the document.

    website_and_docs/content/documentation/webdriver/troubleshooting/errors/_index.zh-cn.md [149-163]

     ## ElementNotVisibleException
     
    -This exception is thrown when the element you are trying to interact with _is_ present in the DOM, but is not visible. 
    +当您尝试与DOM中存在但不可见的元素进行交互时,会抛出此异常。
     
    -### Likely Cause
    +### 可能的原因
     
    -This can occur in several situations:
    -* Another element is blocking your intended element
    -* The element is disabled/invisible to the user
    +这可能发生在以下几种情况:
    +* 另一个元素阻挡了您想要交互的元素
    +* 该元素被禁用或对用户不可见
     
    -### Possible Solutions
    +### 可能的解决方案
     
    -This issue cannot always be resolved on the user's end, however when it can it is usually solved by the following: 
    -using an explicit wait, or interacting with the page in such a way to make the element visible 
    -(scrolling, clicking a button, etc.)
    +这个问题并不总是能在用户端解决,但是当可以解决时,通常可以通过以下方法:
    +使用显式等待,或以某种方式与页面交互使元素可见
    +(滚动、点击按钮等)
     
    • Apply this suggestion
    Suggestion importance[1-10]: 9

    Why: The suggestion to translate the newly added content into Chinese is highly relevant and improves consistency with the rest of the document, which is written in Chinese. This enhances readability and accessibility for the target audience.

    9

    💡 Need additional feedback ? start a PR chat

    @shbenzer
    Copy link
    Contributor Author

    issue #1276

    @shbenzer
    Copy link
    Contributor Author

    @harsha509 what do you think?

    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]: 2
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    1 participant