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

Improved version of #729 without questionable commits #733

Merged
merged 26 commits into from
Aug 27, 2019

Conversation

hmenke
Copy link
Member

@hmenke hmenke commented Aug 27, 2019

I rebased the complete Mo-Gul/master branch onto pgf-tikz/master and dropped the botched merges.

Mo-Gul and others added 25 commits August 27, 2019 11:56
…s in `extract.lua`

- started adding libraries to the codeexamples
  (so far not all needed styles and definitions were added using `pre` key)
…r files fail TeXing using the build bash script.

! `pre` stuff containing (double) hashes fail in automatic testing, because single hashes are needed there
! first `codeexample` in `pgfmanual-en-xxcolor.tex` isn't extracted
  (Is this a bug in `extract.lua`? Are there more `codeexamples` elsewhere that are not extracted?)
! second `codeexample` in pgfmanual-en-xxcolor.tex` shows spurious "-6pt-6pt" in the PDF in front of the `minipage` when TeXing the extracted file
  - incorporated fixes from main PGF repository (provided by Henri)
  - changed `\documentclass` from `article` to `standalone`
  - reordered some stuff
- started switching from `libraries/tikz={...}` to `preamble={\usetikzlibrary{...}}`
…kzlibrary{...}}`

- continued with following files in the manual
- continued adding code to make extracted `codeexample`s work
  (so they can be found more easily)
…unt for the manual files at `/tex/generic/pgf/graphdrawing/lua/pgf/`

- accounted for some more `codeexample`s in `doc/generic/pgf/text-en/`
- started moving `setup code,hidden` to `preamble` of the `codeexample`s
- had a look at the `codeexample`s where a leading space was introduced (see pgf-tikz#711 (comment)). Some of them could be removed but others are introduced because of code added to the `pre` key where I don't have a clue if/how this can be avoided
…y" by default)

  - therefore added loading `graphs` library in `graphs.standard` library
  - adjusted `preamble` code in `codeexample`s accordingly
 - there exist `graphdrawing` `codeexample`s in the manual that don't need the `graphs` library --> adjusted `codeexample`s accordingly
… `pgfmanual-en-base-quick.tex`

Co-Authored-By: Henri Menke <henri@icp.uni-stuttgart.de>
Copy link
Contributor

@Mo-Gul Mo-Gul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I cloned your fork and diffed it with my (local) fork. All my changes were applied successfully and only some stuff that is definitely not from me remained.

Many thanks for taking care!!

@Mo-Gul Mo-Gul merged commit 3ec5292 into pgf-tikz:master Aug 27, 2019
@hmenke hmenke deleted the PimpCodeexamples branch August 28, 2019 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants