Make the "edit this setting" phantom key-bindable

Open
#312 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
25/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
python
Domain
tooling

Research direction

Start by locating the existing on_navigate callback and the phantom href that carries the setting name. Review the Sublime Text 4 command and key-binding behavior, including the optional setting argument and the caret fallback when the current line has no setting. Done means the action can be invoked by a key binding and selects the intended setting in both comment and non-comment contexts.

Written by the indexing model from the issue text.

Description

enhancement

With ST4, we can implement our on_navigate callback as a command and thus also allow people to bind a key to this command. To make this work properly, we would need an optional argument for the setting name (that is passed in the phantom href) with the fallback picking the setting name of the current line.

If there is no setting on the current line, we could either choose the preceding or the following setting, depending on whether the caret is inside a comment currently.

This was raised on Discord.

Dominant language
Python
Stars
455
Forks
86
Avg merge
7d 10h
Merged PRs (30d)
1

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from SublimeText/PackageDev

All issues in SublimeText/PackageDev

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.