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.

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

Open links externally instead of in the html editor in etsdemo application #1446

Merged
merged 2 commits into from
Dec 16, 2020

Conversation

rahulporuri
Copy link
Contributor

This PR updates the etsdemo application such that clicking on links in the description will be opened in the default browser instead of in the html editor itself. This behavior was observed in #1445 and was recommended as the right user experience - #1445 (comment)

instead of in the editor

	modified:   ets-demo/etsdemo/app.py
@rahulporuri rahulporuri added this to In Progress in Enthought OSS Q4 2020 Dec 16, 2020
Copy link
Contributor

@kitchoi kitchoi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Minor nitpick.

ets-demo/etsdemo/app.py Outdated Show resolved Hide resolved
@kitchoi
Copy link
Contributor

kitchoi commented Dec 16, 2020

My bad I did not manually test this with my usual PySide2-from-PyPI exercise.
Thanks to @aaronayres35 testing this. We might need to revert this because this switch makes the entire description screen blank.
Screenshot 2020-12-16 at 16 55 56

kitchoi added a commit that referenced this pull request Dec 16, 2020
@rahulporuri rahulporuri moved this from Done to Sprint 6 : Dec 14 2020 - Jan 2 2021 in Enthought OSS Q4 2020 Jan 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Enthought OSS Q4 2020
Sprint 6 : Dec 14 2020 - Jan 2 2021
Development

Successfully merging this pull request may close these issues.

None yet

3 participants