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

Jump to bottom

proposal: x/tools/present: add support for tasklists #20286

Closed
afh opened this issue May 8, 2017 · 6 comments
Closed

proposal: x/tools/present: add support for tasklists #20286

afh opened this issue May 8, 2017 · 6 comments
Labels
FrozenDueToAge Proposal
Milestone
Proposal

Comments

@afh
Copy link

afh commented May 8, 2017

Proposal

Creating tasklists (a list with a checked or unchecked checkbox) can be helpful, especially in blog post articles or talk slides.

I hereby propose to add the following additional markup to [present.font])https://github.com/golang/tools/blob/master/present/style.go#L40):

  • - [_] to denote a pending task, which will render the marked text within the <del>-tag
  • - [x] to denote a completed task, which will render the marked text within the <ins>-tag

See Gerrit Change #42819 for details.

What version of Go are you using (go version)?

go version go1.8.1 darwin/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Volumes/Users/afh/Library/Go"
GORACE=""
GOROOT="/opt/homebrew/Cellar/go/1.8.1/libexec"
GOTOOLDIR="/opt/homebrew/Cellar/go/1.8.1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/h7/hy79lh5922l19t0v8vjrxn_c0000gp/T/go-build335905375=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
@afh afh changed the title x/tools/present: add support for tasklists to the present package Add support for tasklists to the present package May 8, 2017
@bradfitz bradfitz changed the title Add support for tasklists to the present package proposal: x/tools/present: add support for tasklists May 8, 2017
@gopherbot gopherbot added this to the Proposal milestone May 8, 2017
@ianlancetaylor
Copy link
Contributor

CC @adg

@adg
Copy link
Contributor

adg commented May 9, 2017

It seems like a reasonable addition to me. What do you think @robpike?

@robpike
Copy link
Contributor

robpike commented May 9, 2017

Although reasonable in isolation, I am very nervous of creeping featurism in this tool, which should be bare-bones. It's getting messy. If you need fancy formatting, there are plenty of other tools to do that for you. Present is for simple presentations.

@afh
Copy link
Author

afh commented May 9, 2017

Thanks @robpike for considering it. Actually I'd like to use tasklists in a blog based on x/tools/blog and since that uses x/tools/present I had the idea of adding the support for tasklists there.

Maybe I got a little too excited about the two packages and got carried away not considering the implications it may have.
Then again if present is for simple presentations how come blog depends on it? 😉

@robpike
Copy link
Contributor

robpike commented May 9, 2017

The blog, by design and construction and choice, avoids fanciness too.

@afh
Copy link
Author

afh commented May 10, 2017

I respect your opinion on the matter and consider this rejected then.

@afh afh closed this as completed May 10, 2017
@golang golang locked and limited conversation to collaborators May 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge Proposal
Projects
None yet
Milestone
Proposal
Development

No branches or pull requests
5 participants
@afh @ianlancetaylor @robpike @adg @gopherbot