Tissevert/pandoc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1
pandoc/test/command/3516.md

99 lines
2.1 KiB
Markdown
Raw Normal View History

RST writer: improve grid table output, fix bug with empty rows. Uses the new gridTable in Writers.Shared, which is here improved to better handle 0-width cells. Closes #3516.
2017-03-21 14:16:46 +01:00
Correctly handle empty row:
```
% pandoc -f markdown -t rst
+---+---+
| 1 | 2 |
+---+---+
| | |
+---+---+
^D
+---+---+
| 1 | 2 |
+---+---+
| | |
+---+---+
```
Added to issue 3516 command test to debug test failure on appveyor.
2017-03-22 14:36:12 +01:00
Temporarily added these to figure out what is happening
on Windows builds.
```
% pandoc -f markdown -t native
+---+---+
| 1 | 2 |
+---+---+
| | |
+---+---+
^D
Use pretty-simple to format native output. Previously we used our own homespun formatting. But this produces over-long lines that aren't ideal for diffs in tests. Easier to use something off-the-shelf and standard. Closes #7580. Performance is slower by about a factor of 10, but this isn't really a problem because native isn't suitable as a serialization format. (For serialization you should use json, because the reader is so much faster than native.)
2021-09-19 21:09:51 +02:00
[ Table
Switch from pretty-simple to pretty-show for native output. Update tests. Reason: it turns out that the native output generated by pretty-simple isn't always readable by the native reader. According to https://github.com/cdepillabout/pretty-simple/issues/99 it is not a design goal of the library that the rendered values be readable using 'read'. This makes it unsuitable for our purposes. pretty-show is a bit slower and it uses 4-space indents (non-configurable), but it doesn't have this serious drawback.
2021-09-29 06:17:53 +02:00
( "" , [] , [] )
(Caption Nothing [])
[ ( AlignDefault , ColWidth 5.555555555555555e-2 )
, ( AlignDefault , ColWidth 5.555555555555555e-2 )
Use pretty-simple to format native output. Previously we used our own homespun formatting. But this produces over-long lines that aren't ideal for diffs in tests. Easier to use something off-the-shelf and standard. Closes #7580. Performance is slower by about a factor of 10, but this isn't really a problem because native isn't suitable as a serialization format. (For serialization you should use json, because the reader is so much faster than native.)
2021-09-19 21:09:51 +02:00
]
Switch from pretty-simple to pretty-show for native output. Update tests. Reason: it turns out that the native output generated by pretty-simple isn't always readable by the native reader. According to https://github.com/cdepillabout/pretty-simple/issues/99 it is not a design goal of the library that the rendered values be readable using 'read'. This makes it unsuitable for our purposes. pretty-show is a bit slower and it uses 4-space indents (non-configurable), but it doesn't have this serious drawback.
2021-09-29 06:17:53 +02:00
(TableHead ( "" , [] , [] ) [])
[ TableBody
( "" , [] , [] )
(RowHeadColumns 0)
[]
[ Row
( "" , [] , [] )
[ Cell
( "" , [] , [] )
AlignDefault
(RowSpan 1)
(ColSpan 1)
[ Plain [ Str "1" ] ]
, Cell
( "" , [] , [] )
AlignDefault
(RowSpan 1)
(ColSpan 1)
[ Plain [ Str "2" ] ]
]
, Row
( "" , [] , [] )
[ Cell
( "" , [] , [] ) AlignDefault (RowSpan 1) (ColSpan 1) []
, Cell
( "" , [] , [] ) AlignDefault (RowSpan 1) (ColSpan 1) []
]
]
]
(TableFoot ( "" , [] , [] ) [])
Use pretty-simple to format native output. Previously we used our own homespun formatting. But this produces over-long lines that aren't ideal for diffs in tests. Easier to use something off-the-shelf and standard. Closes #7580. Performance is slower by about a factor of 10, but this isn't really a problem because native isn't suitable as a serialization format. (For serialization you should use json, because the reader is so much faster than native.)
2021-09-19 21:09:51 +02:00
]
Added to issue 3516 command test to debug test failure on appveyor.
2017-03-22 14:36:12 +01:00
```
```
% pandoc -f native -t rst
Implement the new Table type
2020-03-28 23:22:48 +01:00
[Table ("",[],[]) (Caption Nothing
Adapt to the newest Table type, fix some previous adaptation issues - Writers.Native is now adapted to the new Table type. - Inline captions should now be conditionally wrapped in a Plain, not a Para block. - The toLegacyTable function now lives in Writers.Shared.
2020-04-04 22:35:42 +02:00
[])
[(AlignDefault,ColWidth 5.555555555555555e-2)
,(AlignDefault,ColWidth 5.555555555555555e-2)]
(TableHead ("",[],[])
Implement the new Table type
2020-03-28 23:22:48 +01:00
[Row ("",[],[])
Adapt to the removal of the RowSpan, ColSpan, RowHeadColumns accessors
2020-04-06 01:53:35 +02:00
[Cell ("",[],[]) AlignDefault (RowSpan 1) (ColSpan 1)
Implement the new Table type
2020-03-28 23:22:48 +01:00
[]
Adapt to the removal of the RowSpan, ColSpan, RowHeadColumns accessors
2020-04-06 01:53:35 +02:00
,Cell ("",[],[]) AlignDefault (RowSpan 1) (ColSpan 1)
Adapt to the newest Table type, fix some previous adaptation issues - Writers.Native is now adapted to the new Table type. - Inline captions should now be conditionally wrapped in a Plain, not a Para block. - The toLegacyTable function now lives in Writers.Shared.
2020-04-04 22:35:42 +02:00
[]]])
Adapt to the removal of the RowSpan, ColSpan, RowHeadColumns accessors
2020-04-06 01:53:35 +02:00
[(TableBody ("",[],[]) (RowHeadColumns 0)
Adapt to the newest Table type, fix some previous adaptation issues - Writers.Native is now adapted to the new Table type. - Inline captions should now be conditionally wrapped in a Plain, not a Para block. - The toLegacyTable function now lives in Writers.Shared.
2020-04-04 22:35:42 +02:00
[]
[Row ("",[],[])
Adapt to the removal of the RowSpan, ColSpan, RowHeadColumns accessors
2020-04-06 01:53:35 +02:00
[Cell ("",[],[]) AlignDefault (RowSpan 1) (ColSpan 1)
Adapt to the newest Table type, fix some previous adaptation issues - Writers.Native is now adapted to the new Table type. - Inline captions should now be conditionally wrapped in a Plain, not a Para block. - The toLegacyTable function now lives in Writers.Shared.
2020-04-04 22:35:42 +02:00
[Para [Str "1"]]
Adapt to the removal of the RowSpan, ColSpan, RowHeadColumns accessors
2020-04-06 01:53:35 +02:00
,Cell ("",[],[]) AlignDefault (RowSpan 1) (ColSpan 1)
Adapt to the newest Table type, fix some previous adaptation issues - Writers.Native is now adapted to the new Table type. - Inline captions should now be conditionally wrapped in a Plain, not a Para block. - The toLegacyTable function now lives in Writers.Shared.
2020-04-04 22:35:42 +02:00
[Para [Str "2"]]]
,Row ("",[],[])
Adapt to the removal of the RowSpan, ColSpan, RowHeadColumns accessors
2020-04-06 01:53:35 +02:00
[Cell ("",[],[]) AlignDefault (RowSpan 1) (ColSpan 1)
Adapt to the newest Table type, fix some previous adaptation issues - Writers.Native is now adapted to the new Table type. - Inline captions should now be conditionally wrapped in a Plain, not a Para block. - The toLegacyTable function now lives in Writers.Shared.
2020-04-04 22:35:42 +02:00
[]
Adapt to the removal of the RowSpan, ColSpan, RowHeadColumns accessors
2020-04-06 01:53:35 +02:00
,Cell ("",[],[]) AlignDefault (RowSpan 1) (ColSpan 1)
Adapt to the newest Table type, fix some previous adaptation issues - Writers.Native is now adapted to the new Table type. - Inline captions should now be conditionally wrapped in a Plain, not a Para block. - The toLegacyTable function now lives in Writers.Shared.
2020-04-04 22:35:42 +02:00
[]]])]
(TableFoot ("",[],[])
[])]
Added to issue 3516 command test to debug test failure on appveyor.
2017-03-22 14:36:12 +01:00
^D
+---+---+
| 1 | 2 |
+---+---+
| | |
+---+---+
```
Reference in a new issue Copy permalink