Closed shbenzer closed 2 weeks ago
Visit the deploys page to approve it
Name | Link |
---|---|
Latest commit | 63854b51e5a3d2032beb5edcfe7eba89da8fe4f8 |
β±οΈ Estimated effort to review: 1 π΅βͺβͺβͺβͺ |
π§ͺ No relevant tests |
π No security concerns identified |
β‘ No key issues to review |
Category | Suggestion | Score |
Enhancement |
Replace placeholder code with actual JavaScript and Kotlin examples___ **Consider adding actual JavaScript and Kotlin code examples instead of using the{{< badge-code >}} shortcode. This would provide more value to users looking for specific implementation details in these languages.** [website_and_docs/content/documentation/webdriver/elements/finders.en.md [139-143]](https://github.com/SeleniumHQ/seleniumhq.github.io/pull/1897/files#diff-e21fb1eabd6a7eaf605f8f733ad4e5406d5ad60908a37a6da9b2c363c021ceaeR139-R143) ```diff -{{< tab header="JavaScript" text=true >}} -{{< badge-code >}} +{{< tab header="JavaScript" >}} +const shadowHost = await driver.findElement(By.css('#shadow_host')); +const shadowRoot = await shadowHost.getShadowRoot(); +const shadowContent = await shadowRoot.findElement(By.css('#shadow_content')); {{< /tab >}} -{{< tab header="Kotlin" text=true >}} -{{< badge-code >}} +{{< tab header="Kotlin" >}} +val shadowHost = driver.findElement(By.cssSelector("#shadow_host")) +val shadowRoot = shadowHost.getShadowRoot() +val shadowContent = shadowRoot.findElement(By.cssSelector("#shadow_content")) {{< /tab >}} ``` - [ ] **Apply this suggestion** Suggestion importance[1-10]: 9Why: The suggestion to replace the placeholder `{{< badge-code >}}` with actual JavaScript and Kotlin code examples is highly beneficial. It provides concrete examples that enhance the documentation's value by offering specific implementation details, which is crucial for users seeking practical guidance. | 9 |
User description
Made improvements to Shadow-Dom section: made ruby code of type code instead of text, added badge codes to missing exampes
Description
added badge codes to missing examples in shadow dom sections made ruby code of type code instead of type text
Motivation and Context
increases readability
Types of changes
Checklist
PR Type
documentation
Description
code
instead oftext
across multiple language documentation files.Changes walkthrough π
finders.en.md
Update code formatting and add badges in English documentation
website_and_docs/content/documentation/webdriver/elements/finders.en.md
finders.ja.md
Update code formatting and add badges in Japanese documentation
website_and_docs/content/documentation/webdriver/elements/finders.ja.md
finders.pt-br.md
Update code formatting and add badges in Portuguese documentation
website_and_docs/content/documentation/webdriver/elements/finders.pt-br.md
finders.zh-cn.md
Update code formatting and add badges in Chinese documentation
website_and_docs/content/documentation/webdriver/elements/finders.zh-cn.md