Changed srcElement to target in mini browser

[Tpuljak]

Signed-off-by: Toma Puljak toma.puljak@hotmail.com

What it does

The following pull-request updates the deprecated scrElement event prop with target for the mini-browser package when handling the ENTER key to reload the page and update the content to the new location. The change is suggested by mozilla documentation.

How to test

In order to test the changes, you can use the Open URL command with some minor tweaks. Since the Open URL command uses a readonly input, it will need to be updated to show so the handler can successfully process the ENTER key code.

1. update the following to show:

theia/packages/mini-browser/src/browser/mini-browser-open-handler.ts

Line 282 in deccec6

toolbar: 'read-only',

2. rebuild the mini-browser extension using: npx run build @theia/mini-browser
3. rebuild the example application (ex: npx run build @theia/example-electron)
4. start the application, and using the command palette (F1) trigger the Open URL command
5. input a URL (ex: a file on your filesystem that can be previewed such as REAMDE.md)
6. ensure that the input can be updated, and upon pressing ENTER the content is refreshed and maps to the new location:

Review checklist

• as an author, I have thoroughly tested my changes and carefully followed the review guidelines

Reminder for reviewers

• as a reviewer, I agree to behave in accordance with the review guidelines

[vince-fugnitto]

@Tpuljak thank you for your first contribution!

Before we can review your changes please be sure to complete the following:

• Please be sure to add a proper sign-off to your commit
• Please be sure to sign and complete the Eclipse Contributor Agreement (ECA) with the same email as your sign-off.
• Please be sure to update your pull-request description with a how to test section which is currently omitted. This section helps reviewers properly understand and test your changes.

[Tpuljak]

@vince-fugnitto I've completed everything you asked for 😄

[DucNgn]

The PR looks good to me 👍 . I believe more information about this change could be found here

[vince-fugnitto]

The changes work well 👍
With a little bit of changes for the how to test I was able to confirm that the handleInputChange handler correctly maps the location when ENTER is triggered: