wordpress-mobile / wordpress-mobile/AztecEditor-iOS
hr element cannot be contained in p element
Nobody has claimed this yet.
- Dominant language
- Swift
- Stars
- 669
- Forks
- 153
- Avg merge
- 11h 27m
- Merged PRs (30d)
- 4
Description
Expected behavior
hr element is self-contained element, should not be contained in p element.
Actual behavior
hr element is contained in p element, this may cause some issues:
-
The editor automatically adds an empty P element before the HR element each time if hr element is contained in p element.
-
If the hr element not contained in p element, the editor will add p element surround hr element automatically.
Steps to reproduce the behavior
- Open the editor with the content that saved before
- Insert hr element
- save the html content.
- repeat step 1-3
Tested on iPhone 7, iOS 12.1.2, Aztec 1.4.1
References:
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the saved-HTML cycle in Aztec 1.4.1 on an iOS editor, focusing on how HR and P elements are handled. Done means inserting and saving an HR does not add an empty P before it or wrap it in a P on the next edit.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- html, swift
- Domain
- mobile
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100