Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-46497

Atto: clicking RTL button and then LTR button, wrap the selected text with redundant P html elements.

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Make sure you are using Atto editor (User's preferences) and that you are using the English UI language.
      2. Navigate to: Site administration ► Plugins ► Text editors ► Atto HTML editor ► Atto toolbar settings
      3. Inside "Toolbar config", add ",rtl" to "align = align" (so it looks like "align = align,rtl"). And save settings. (So now you have two new buttons on the Atto's second/hidden toolbar - RTL/LTR)
      4. Navigate into a page where you can use the HTML editor (You can start a Discussion on a Forum or Edit your own user's description on your profile page)
      5. Type into the Atto editor a few words (You can paste: "אני כותב בעברית And also in English וחוזר לכתוב בעברית")
      6. Now, click the "" Atto toolbar button and then click the "" button. And repeat it 2-3 times.
      7. Switch to HTML mode (use the "<>" icon on the Atto's second/hidden toolbar)
      8. Make sure there are no redundant HTML P elements surrounding your text. (As seen on the attached screen capture)
      Show
      Make sure you are using Atto editor (User's preferences) and that you are using the English UI language. Navigate to: Site administration ► Plugins ► Text editors ► Atto HTML editor ► Atto toolbar settings Inside "Toolbar config", add ",rtl" to "align = align" (so it looks like "align = align,rtl"). And save settings. (So now you have two new buttons on the Atto's second/hidden toolbar - RTL/LTR) Navigate into a page where you can use the HTML editor (You can start a Discussion on a Forum or Edit your own user's description on your profile page) Type into the Atto editor a few words (You can paste: "אני כותב בעברית And also in English וחוזר לכתוב בעברית") Now, click the " " Atto toolbar button and then click the " " button. And repeat it 2-3 times. Switch to HTML mode (use the "<>" icon on the Atto's second/hidden toolbar) Make sure there are no redundant HTML P elements surrounding your text. (As seen on the attached screen capture)
    • Affected Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-46497_master

      Description

      Clicking RTL button and then LTR button, several times, wrap the selected text with redundant P html elements that include inline directionality settings. (see attached screen capture)
      It should change the immediate selected's text P or SPAN or DIV element "dir" attribute. And not wrap the element with any other element. (See TinyMCE of desired behavior)

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                3 Vote for this issue
                Watchers:
                13 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/Nov/15