Am I correct that this "bug" can only be observed if you write the tags manually and are not using the editor-xtd plugin

Brian, this bug (without quotes) can be observed independently of the way of creating tags. I use Code Mirror, and it insertes the title after the class attribute.

I ask this because it is important for testers to know how to recreate the
issue

On 8 June 2016 at 09:31, Philip Sorokin notifications@github.com wrote:

Brian, this bug (without quotes) can be observed independently of the way
of creating tags. I use Code Mirror, and it insertes the title after the
class attributes.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#10754 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8Xk3Hj0eK61pj4MY9T0Efag_UaaYks5qJn3KgaJpZM4IwcQ0
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

I noticed that making the self-closing slash optional is not working. So, I have edited this PR.

I just tested and I cannot replicate the problem
1. Tested with tinymce pagebreak - no problem
2. Tested with codemirror pagebreak - no problem (the code generated is the same as above)
3. Tested pasting pagebreak code manually in the "wrong" order - no problem

Please see the video for tests 2 and 3
https://www.dropbox.com/s/aalqneftgq9222r/10754.mp4?dl=0

@brianteeman, the alias is correct in any case, while the meta tag <title>Is not changing</title> if the order of attributes is as described above.

I have tested this item ✅ successfully on 51e1172

I see what you mean now and the PR fixes that issue - however the only way I can get the code in the wrong order is by entering it manually. The editor plugin always does it correctly

As far, as I know, Code Mirror always insertes page breaks like this: <hr class="system-pagebreak" title="title" alt="description" />. This order breaks the title. However, it is correct when I put the title and alias attributes manually before the class attribute.

Isnt it the other way around

On 8 June 2016 at 19:03, Philip Sorokin notifications@github.com wrote:

As far, as I know, Code Mirror always insertes page breaks like this: class="system-pagebreak" title="title" alt="description" />. This order
breaks the title. However, it is correct when I put the title and alias
attributes manually before the class attribute.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#10754 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8RzWvHIZDghsXmZrbGqj3Rw1Igrjks5qJwP6gaJpZM4IwcQ0
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

I have checked again, and it is not.

This PR has received new commits.

CC: @brianteeman

Will it be a part of Joomla! 3.6?

I have tested this item ? unsuccessfully on 7d05595

Tested according to instructions, using CodeMirror.
Set alternative title attribute before class attribute (step 3): working as described in instructions.
Set alternative title after classname attribute (step 4): not working as described in instructions: alt title is set.
Try to create page break without a self closing slash (step I-1): fails, and still fails after applying patch.

@addondev please can you check the failure report above

@brianteeman
This PR requires more work. So, it should be closed. I will open a new one.