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.

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

[WIP] actions: add Tagline command #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

josephholsten
Copy link
Contributor

@josephholsten josephholsten commented Jul 1, 2016

This is totally not ready to merge. The desire of this is to replace the ~/.de/tagline with a command which can live in ~/.de/startup. It doesn't do that yet, and it has other problems besides.

  • ResetTagline should use the value from Tagline instead of getDefaultTagline
  • Tagline should accept args containing a space, eg Tagline:Foo Bar. Perhaps quoting/escaping (ugh)? Can we just rely on startup to use complete lines, and use selection for manual execution?

@josephholsten josephholsten changed the title actions: add Tagline command [WIP] actions: add Tagline command Jul 1, 2016
@driusan
Copy link
Owner

driusan commented Jul 2, 2016

Colons get replaced by space (unless they're escaped with a backslash) to make it possible to have spaces in arguments to executable words in the tagline.

I think that the appropriate behaviour for the startup script should be to execute lines one at a time as you suggest, even if there's spaces, but I'd have to look at the code to see if that's currently the case.

@josephholsten
Copy link
Contributor Author

looks like a ~/.de/startup file of:

Tagline:Save Discard Cut Cupy Paste Undo Exit Shell a+ a-
TermWidth:72

sets the tagline to the entire string correctly, and still correctly uses the TermWidth command.

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.

None yet

2 participants