Add missing isVisible command in new Element API

[garg3133]

Description of the bug/issue

The isVisible command is currently missing from the new Element API. To support this command, a new file named isVisible.js needs to be added inside lib/api/web-element/commands directory, whose content should be similar to the get* command files present in the same directory.

The argument passed to this.runQueuedCommandScoped in these files is the name of the corresponding method to be called from the lib/transport/selenium-webdriver/method-mappings.js file.

Moreover, we'd also need to have isDisplayed() as an alias to the isVisible() command.

Steps to reproduce

Go to any test file and try to add const result = await browser.element('<selector>').isVisible() and then run the test. This would result in an error right now, while if you replace isVisible() with getText(), it should work fine.

After adding the command implementation, the .isVisible() command should work fine and return a boolean promise.

TODO

• Add missing command
• Add isDisplayed as an alias to isVisible() command
• Fix JSDoc of the command
• Add tests for the new command
• Add types for the new command

[FireNdIce3]

Can I take this issue?
Thanks

[subhajit20]

hey @garg3133 can I get assigned for this issue?

[starvader13]

Hey @garg3133 , I can take this issue and fix it up. Can you assign this to me?

[PiyushThapaa]

@garg3133 Hey, I would like to work on it can you assign it to me...

[karan2198]

@garg3133 hello, I'm new to open source and eager to contribute. Could you please assign this issue to me?

[rumbleroii]

Hey, I would like to work on this issue. Could you assign it to me?

[Vaaruna2709]

Can I work on this issue?

[Vaaruna2709]

Could you please review my PR?

[dikwickley]

@Vaaruna2709 I think you have raised PR to the main branch of your own fork.

[srs-sudeep]

Hey, @garg3133 , I would like to work on it can you assign it to me...

[26vanshika]

Hey @garg3133, I would like to work on this can you assign me

[Ayush-Vish]

@garg3133 I have worked on this issue and tested the isVisible and isDisplayed Commands.
Below is the Video for it.

2024-02-26.17-28-35.mp4

Please assign me this issue.
I just made two files which are isVisible and isDisplayed and both work as accepted

[Jai0401]

Hi @garg3133 ,

I've identified that the isVisible command is missing in the new Element API, causing errors when users try to use it. To address this, I've proposed a solution by creating a new file named isVisible.js, which includes the missing command and introduces a helpful alias isDisplayed. The implementation follows the existing commands' structure, including clear JSDoc. Could I take on this issue to contribute to the fix?

Thanks!

I am attaching some screenshots:

[Honey2339]

@garg3133 I've taken a look at issue and would like to contribute by solving it. It seems like a good issue to start off with. Looking forward to your guidance.

[sabinonweb]

Hey @garg3133 can i take this issue? Will like to contriburte,please assign it to me.

[garg3133]

Fixed in #4039