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.

Sign up for GitHub

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

Jump to bottom

Hidden setting to override the text editor. #246

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Hidden setting to override the text editor. #246

wants to merge 2 commits into from

Commits on Jan 30, 2024

  1. Hidden setting to override the text editor. 

    Note that people who run gtimelog straight from Git will need to
manually run ``make`` to recompile the gsettings schema, or gtimelog
will segfault when you press Ctrl+E.

Also note that using `gsettings` is a pain when the schema is not
installed globally, but you can use

    dconf write /org/gtimelog/editor '"gvim %s"'

and then tweak it with dconf-editor.

Error handling is missing: if you set editor to gvim and gvim is not
installed, you'll see a Python traceback on the console.

Application.prepare_args() is a separate static method so I could write
unit tests for it, but I haven't done so.

Closes #113.
    @mgedmin
    mgedmin committed Jan 30, 2024
    Configuration menu
    Copy the full SHA
    50e3f4b View commit details
    Browse the repository at this point in the history

  2. Try to report editor execution errors 

    The error is only visible in report mode.
    @mgedmin
    mgedmin committed Jan 30, 2024
    Configuration menu
    Copy the full SHA
    4ef4596 View commit details
    Browse the repository at this point in the history