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

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

Comments

@afh
Copy link

@afh 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
@gopherbot gopherbot added the Proposal label May 8, 2017
@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented May 8, 2017

CC @adg

@adg
Copy link
Contributor

@adg adg commented May 9, 2017

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

@robpike
Copy link
Contributor

@robpike 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 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 robpike commented May 9, 2017

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

@afh
Copy link
Author

@afh afh commented May 10, 2017

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

@afh afh closed this 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.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.