cpython icon indicating copy to clipboard operation
cpython copied to clipboard

bpo-32548: IDLE: Convert labels to hyperlinks in help_about

Open csabella opened this issue 7 years ago • 10 comments

In the help_about module of IDLE, convert text that contained URLs to be clickable links.

https://bugs.python.org/issue32548

csabella avatar Jan 14 '18 01:01 csabella

@ned-deily Thank you for your review. I had seen that code in EditorWindow and I wasn't sure if I should include it in this PR or not. The URL link for the docs already existed on the Help -> About screen and I just changed it from being plain text to being blue, underlined, and clickable. I was attempting to introduce the functionality [of a hyperlink] without changing the dialog too much in order to make the PR more reasonable to review.

Do you think that your request should be introduced here or in a subsequent PR?

Thanks!

csabella avatar Jan 14 '18 10:01 csabella

The EditorWindow.python_docs method is linked to the 'Python Docs F1' entry on the Help menu. (The function, the help_url code wrapped as a function, and the link code should be moved to the help module.) The 'IDLE Help' entry is linked to a no-trailing-whitespace copy of the IDLE .html page in the docs. (Would 'IDLE Doc' be clearer?) The point of the copy is to avoid an online dependency.

The link in About IDLE, to the online copy of the IDLE .html page, is now redundant with the latter and perhaps should be removed. Or it should open the local copy, the same as IDLE Help'. It was less redundant when 'IDLE Help' linked to a separate plain text within idlelib.

Before adding idlelib/help.html and the custom html parser, I looked into opening the IDLE page within a possibly off-line system-dependent copy of the docs, and preferred the completely self-contained solution. I would still like to have making the filtered copy be part of the release process, but that is a separate issue.

terryjreedy avatar Jan 15 '18 02:01 terryjreedy

@terryjreedy I removed the doc link for the About page, so you could see what it looked like. I also changed create_link to take a callback instead of just a link so that it would be easier to change the buttons to use it. Based on your comments about using online browser, I added a message box to display_browser_link in case the browser couldn't be opened. I'm not sure if there's a better way to include the link to the webpage for the Email List.

csabella avatar Jan 16 '18 23:01 csabella

The mailing list link works fine, but now that I am effectively the list owner, I want to edit the page before sending people there. I also want to drop other buttons or convert to links also. I decided not to rush this for 3.7.0rc1.

terryjreedy avatar May 20 '18 02:05 terryjreedy

Well, that was unintentional. :-( I was trying to clean up branches and didn't know which ones were still open. I'll need to figure out how to recreate this.

csabella avatar Jun 04 '18 17:06 csabella

@csabella If the branch had been pushed to your GitHub clone in the cloud, you may be able to undelete the branch by going to your GitHub clone page and clicking on the Branches page. Good luck!

ned-deily avatar Jun 04 '18 17:06 ned-deily

I gather that you deleted the branch on your clone that was the basis for the PR and github closed the PR. I have been wondering if that would be the result of such a deletion ;-).

I cannot reopen, which I expected. I can create a local branch from the PR, which I mostly expected because the Files changed button still works to show the diff. And yes, I still intend to review this.

terryjreedy avatar Jun 04 '18 17:06 terryjreedy

Thanks, @ned-deily and @terryjreedy. I was able to recover this one and the other PR by following the second response at: https://stackoverflow.com/questions/33656729/how-to-recover-a-deleted-remote-branch

Luckily, it was actually not too difficult.

What I done to cause the issue: git push origin --delete branch_name

I built a new computer last week, so I had recreated my local clone. The git branch was so nice and uncluttered that I thought I'd do the same on my csabella origin remote at github. Lesson of the day: if you do that on open PRs, it deletes the branch and closes the PR. :-) At least github had the commits, so I was able to use that to recreate the branch. :-)

csabella avatar Jun 04 '18 18:06 csabella

This PR is stale because it has been open for 30 days with no activity.

github-actions[bot] avatar Aug 14 '22 00:08 github-actions[bot]

I have since tested the mail url and it works. However, I think I would like to have that button open a dialog that says something about how to report bugs and ideas.

terryjreedy avatar Aug 14 '22 00:08 terryjreedy

This PR is stale because it has been open for 30 days with no activity.

github-actions[bot] avatar Feb 25 '23 00:02 github-actions[bot]