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.

Sign up for GitHub

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

Jump to bottom

BUG: Text transformation issue with text-transform: capitalize; using Selenium WebDriver with Firefox 127.0.2 #2185

Open
Usielrivas opened this issue Jul 8, 2024 · 4 comments
Open

BUG: Text transformation issue with text-transform: capitalize; using Selenium WebDriver with Firefox 127.0.2 #2185

Usielrivas opened this issue Jul 8, 2024 · 4 comments
Labels
firefox selenium

Comments

@Usielrivas
Copy link

Description:
There is an issue with the text-transform: capitalize; CSS property when using Selenium WebDriver with Firefox 127.0.2. The text transformation does not handle accented characters correctly, resulting in incorrect capitalization.

Steps to Reproduce:

  1. Install Node.js.
  2. Create a new project and install Selenium WebDriver. 
    npm init -y
npm install selenium-webdriver // last version
  3. Download geckodriver 0.34 and place it in a specific directory.
  4. Create the following JavaScript script (selenium_example.js):
const { Builder, By, until } = require('selenium-webdriver');
const firefox = require('selenium-webdriver/firefox');
const path = require('path');

const geckodriverPath = path.join(process.env.HOME, 'gecko/geckodriver');  // Adjust this path

(async function example() {
    let options = new firefox.Options();
    options.setBinary(geckodriverPath);

    let driver = new Builder()
        .forBrowser('firefox')
        .setFirefoxOptions(options)
        .build();

    try {
        const url = 'file:///home/user/test/bug/index.html';

        await driver.get(url);

        await driver.wait(until.elementLocated(By.tagName('h1')), 10000);

        let element = await driver.findElement(By.tagName('h1'));

        let text = await element.getText();

        console.log(`response: ${text}`);
    } finally {
        await driver.quit();
    }
})();
  1. Create the following HTML file (index.html):
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Bug Test</title>
    <style>
        h1 {
            text-transform: capitalize;
        }
    </style>
</head>
<body>
    <h1>Fecha de expiración</h1>
</body>
</html>
  1. Run the script: 
    node selenium_example.js

Expected Behavior:
The output should be:

response: Fecha De Expiración

Actual Behavior:
The output is:

response: Fecha De ExpiraciÓN

Additional Information:

  • Operating System: Fedora 40
  • Node.js: v20.12.2
  • Selenium WebDriver: 4.22.0
  • geckodriver: 0.34.0 (c44f0d09630a 2024-01-02 15:36 +0000)
  • Firefox: 127.0.2

This issue started occurring after updating to Firefox 127.0.2. It appears that the text-transform: capitalize; CSS property does not handle accented characters correctly.

Console Output:
No errors in the console.
@Usielrivas Usielrivas changed the title Text transformation issue with text-transform: capitalize; using Selenium WebDriver with Firefox 127.0.2 BUG: Text transformation issue with text-transform: capitalize; using Selenium WebDriver with Firefox 127.0.2 Jul 8, 2024
@whimboo
Copy link
Collaborator

whimboo commented Jul 17, 2024

In Firefox 127 we landed a fix for https://bugzilla.mozilla.org/show_bug.cgi?id=1888004. That included an update of the Selenium atoms for 'getVisibleText' to revision 33c6b7841a59aaaad55744909c0600f066fd5593. This update includes the changes from SeleniumHQ/selenium#13792.

@diemol could your patch have caused a regression for this atom and behavior for text-transform: capitalize?

@diemol
Copy link

diemol commented Jul 17, 2024

But @whimboo... I applied the patch because you asked for it 🤣

SeleniumHQ/selenium#13792

So I do not know; tests were passing, so I think it was all good. Did we miss something?

@whimboo
Copy link
Collaborator

whimboo commented Jul 17, 2024 •
edited
Loading

There might have been a misunderstanding at that time, and I apologize for that. The change I proposed was only relevant to my case, and I didn't test any other scenarios. I assume there is no test in the Selenium code base that checks for accented characters, which would have caught this regression.

As it looks like we should file a new issue on the Selenium repository to get this issue fixed. @Usielrivas because you found the issue would you like to do that?

@Usielrivas
Copy link
Author

Fact reported in selenium SeleniumHQ/selenium#14271

@whimboo whimboo mentioned this issue Jul 17, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
firefox selenium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@whimboo @diemol @Usielrivas