#80 Navigation features

Đã hợp nhất
herby đã nhập 3 commit từ mmontone/master vào [3]s 2 năm trước cách đây

Hello.

I've implemented some navigation features that I found handy.

  • Find references commands in menus.
  • Browse it supports method names.
  • Open panel menu on right click.
  • Browse selected method from debugger call stack.

Feel free to do whatever you want with these. Modify them, or discard them completely.

Cheers!

Hello. I've implemented some navigation features that I found handy. * Find references commands in menus. * Browse it supports method names. * Open panel menu on right click. * Browse selected method from debugger call stack. Feel free to do whatever you want with these. Modify them, or discard them completely. Cheers!
Herby Vojčík đã nhận xét 2 năm trước cách đây
Chủ sở hữu

Are the methods in other Helios command classes also in 'as yet unclassified' protocol? If not, please, categorize your ones, too. :-)

(and if possible, not by other commits, but fixing old one via rebase)

Are the methods in other Helios command classes also in 'as yet unclassified' protocol? If not, please, categorize your ones, too. :-) (and if possible, not by other commits, but fixing old one via rebase)
Herby Vojčík đã nhận xét 2 năm trước cách đây
Chủ sở hữu

Re: browseIt change - I would like not to have it there... browseIt is the same as printIt or inspectIt - it should first evaluate the selection, then try to show the result of evaluation in browser.

After all, I presume Ctrl+click does this job, so browseIt can stay with its 'evaluate and browse the result' semantics.

(speaking of Ctrl+click: amber/amber#1263)

Re: browseIt change - I would like not to have it there... browseIt is the same as printIt or inspectIt - it _should_ first evaluate the selection, then try to show the result of evaluation in browser. After all, I presume Ctrl+click does this job, so browseIt can stay with its 'evaluate and browse the result' semantics. (speaking of Ctrl+click: https://lolg.it/amber/amber/issues/1263)
Herby Vojčík đã nhận xét 2 năm trước cách đây
Chủ sở hữu

If you added context menu on right-click on every pane (not sure, probably yes), then add "Fixes #11" to the commit message.

If you added context menu on right-click on every pane (not sure, probably yes), then add "Fixes #11" to the commit message.
mmontone đã nhận xét 2 năm trước cách đây
Poster

I've reverted the "browseIt" change and categorized the methods in new commands.

I've done it all with new commits, though. As my git rebase abilities are null but for the most basic changes at the moment :/ .

And so, I couldn't change the commit message for the right click on panels commit (yes, right click opens panels menu for every panel, when an item is selected). (perhaps you can reference the commit from the issue instead?)

Sorry, this is the best I can do at the moment.

I've reverted the "browseIt" change and categorized the methods in new commands. I've done it all with new commits, though. As my git rebase abilities are null but for the most basic changes at the moment :/ . And so, I couldn't change the commit message for the right click on panels commit (yes, right click opens panels menu for every panel, when an item is selected). (perhaps you can reference the commit from the issue instead?) Sorry, this is the best I can do at the moment.
mmontone đã nhận xét 2 năm trước cách đây
Poster

I think I've managed to rebase and apply everything you've requested now.

I think I've managed to rebase and apply everything you've requested now.
Herby Vojčík đã nhận xét 2 năm trước cách đây
Chủ sở hữu

Nice.

One more small thing pls: the class level menuLabel methods are still not classified. :-) Could you fix (and get it back to the original commit as the fixup)? Thanks.

Nice. One more small thing pls: the class level `menuLabel` methods are still not classified. :-) Could you fix (and get it back to the original commit as the fixup)? Thanks.
mmontone đã nhận xét 2 năm trước cách đây
Poster

Done

Done
Herby Vojčík đã nhận xét 2 năm trước cách đây
Chủ sở hữu

Since helios is hardly separable from Amber, add yourself into https://lolg.it/amber/amber/src/master/CONTRIBUTORS, pls; via PR.

Since helios is hardly separable from Amber, add yourself into https://lolg.it/amber/amber/src/master/CONTRIBUTORS, pls; via PR.
Herby Vojčík đã nhận xét 2 năm trước cách đây
Chủ sở hữu

It seems the "Fixes #11" should have been added to the main line... gogs didn't get it from the literary part. :-(

It seems the "Fixes #11" should have been added to the main line... gogs didn't get it from the literary part. :-(
Yêu cầu kéo này đã được sáp nhập thành công!
Đăng nhập để tham gia bình luận.
Không có nhãn
Không có Milestone
Không có người được phân công
2 tham gia
Đang tải...
Hủy bỏ
Lưu
Ở đây vẫn chưa có nội dung nào.