Skip to content

gh-74706: IDLE: Add navigate bar and replace current goto dialog#1884

Closed
mlouielu wants to merge 5 commits intopython:mainfrom
mlouielu:navigatebar
Closed

gh-74706: IDLE: Add navigate bar and replace current goto dialog#1884
mlouielu wants to merge 5 commits intopython:mainfrom
mlouielu:navigatebar

Conversation

@mlouielu
Copy link
Contributor

@mlouielu mlouielu commented May 31, 2017

@mention-bot
Copy link

@mlouielu, thanks for your PR! By analyzing the history of the files in this pull request, we identified @kbkaiser, @terryjreedy and @tiran to be potential reviewers.

return "break"
text.mark_set("insert", "%d.0" % lineno)
text.see("insert")
from idlelib.singlebar import NavigateBar
Copy link
Member

Choose a reason for hiding this comment

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

You named the file navigatebar

Copy link
Member

@terryjreedy terryjreedy left a comment

Choose a reason for hiding this comment

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

As I will explain on the issue, I do not like the concept and am inclined to reject this.

  • a bar with no title is confusing.

@brettcannon
Copy link
Member

To try and help move older pull requests forward, we are going through and backfilling 'awaiting' labels on pull requests that are lacking the label. Based on the current reviews, the best we can tell in an automated fashion is that a core developer requested changes to be made to this pull request.

If/when the requested changes have been made, please leave a comment that says, I have made the requested changes; please review again. That will trigger a bot to flag this pull request as ready for a follow-up review.

@github-actions
Copy link

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

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Feb 19, 2022
@erlend-aasland
Copy link
Contributor

As I will explain on the issue, I do not like the concept and am inclined to reject this.

@terryjreedy, is this still your stance? If so, I'd like to mark this as pending close.
@mlouielu, are you still interested in pushing these changes?

@erlend-aasland erlend-aasland changed the title bpo-30521: IDLE: Add navigate bar and replace current goto dialog gh-30521: IDLE: Add navigate bar and replace current goto dialog Jun 29, 2022
@erlend-aasland erlend-aasland changed the title gh-30521: IDLE: Add navigate bar and replace current goto dialog gh-74706: IDLE: Add navigate bar and replace current goto dialog Jun 29, 2022
@github-actions github-actions bot removed the stale Stale PR or inactive for long period of time. label Jun 30, 2022
@mlouielu mlouielu mannequin mentioned this pull request Apr 10, 2022
@terryjreedy
Copy link
Member

I will try my alternate plan first (see issue), and even that is low priority. (I believe I have not heard from Louie since Sept. 2017.)

@terryjreedy terryjreedy closed this Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants