Results 91 comments of Mike
trafficstars

If you add the raise parameter to Execute function, as I demonstrated use for in Navigate function [here](https://github.com/uezo/TinySeleniumVBA/issues/40#issuecomment-1002866445), then IsElementPresent can be simplified as below: ``` Public Function IsElementPresent(by_ As...

Suggest to change method name IsElementPresent to IsPresent to be consistent with IsEnabled and IsDisplayed..

Excellent work @ezagdd! I suggest to consider changing in all of the above: ``` Optional millisecond ``` to ``` Optional ByVal millisecond ``` **In fact, I went through the entire...

Thanks @ezagdd. Shouldn't sessionId parameter be passed to all subordinate methods (Focus, FindChildElements, and ExecuteScript)? Also the optdata dictionary is not used (not needed?) in SelectByIndex...

same issues with DeSelectByIndex...

Also, sessionId needs to be passed to IsElementSelected and Click in SelectByValue and SelectByVisibleText. Same with FindElements in SelectTagsOptions. FindChildElements and IsElementSelected in AllSelectedOptions. FindChildElements and IsElementSelected in SelectedOptionText IsElementSelected...

Awesome thanks for your effort!

Also consider changing return type of IsElementSelected to boolean?

Also suggest to rename method IsElementSelected to IsSelected... (?)

@ezagdd, I am not sure I understand what you are asking... From above If raise=False then Set Execute = Resp("value"). So can't the caller take action from that? An example...