Repeaters affect cursor position

[JacobEvelyn]

Correct me if I'm wrong, but I believe this is abnormal behavior:

pdf = Prawn::Document.new(margin: 100)
puts "Cursor before regular repeater: #{pdf.cursor}"
pdf.repeat(:all) do
  pdf.canvas do
  end
end
puts "Cursor after regular repeater: #{pdf.cursor}"

pdf = Prawn::Document.new(margin: 100)
puts "Cursor before dynamic repeater: #{pdf.cursor}"
pdf.repeat(:all, dynamic: true) do
  pdf.canvas do
  end
end
puts "Cursor after dynamic repeater: #{pdf.cursor}"

Output:

Cursor before regular repeater: 592.0
Cursor after regular repeater: 692.0

Cursor before dynamic repeater: 592.0
Cursor after dynamic repeater: 592.0

Shouldn't both dynamic and non-dynamic repeaters leave the cursor at 592.0 after they're defined?

[JacobEvelyn]

Is this behavior correct?

[practicingruby]

Hi, we did a bulk close on issues that did not see activity in the last month or so, because the project has been mostly unmaintained for the last several months.

I made an announcement on the Prawn mailing list about this, with the intention of re-opening any ticket that still had someone interested in it.

I've re-opened this one and removed the 'stale' tag, and will investigate as I work through the review queue. Thanks!

[JacobEvelyn]

Ah, thanks for the explanation.

[practicingruby]

I've confirmed that the issue above exists in master, and I agree, it's the wrong behavior. Does someone want to investigate this one?

[practicingruby]

@JacobEvelyn: I think this may actually be a bug in canvas... I'll dig deeper and see what I can find.

[practicingruby]

@JacobEvelyn: Can you try master and see if it fixes your issue?

[practicingruby]

I believe this ought to solve your problem, but please leave a comment if it does not and I will re-open.