gocodebox / lifterlms

LifterLMS, a WordPress LMS Solution: Easily create, sell, and protect engaging online courses.
https://lifterlms.com
GNU General Public License v3.0
181 stars 135 forks source link

Acess Plan editor: Text mode still acts as Visual mode upon creation #1562

Open eri-trabiccolo opened 3 years ago

eri-trabiccolo commented 3 years ago

Reproduction Steps

Expected Behavior

Actual Behavior

If you save the access plan the Text mode then displays the HTML but the toolbar isn't visible. Refreshing the page solves the issue. So let's say this is the workaround.

Error Messages / Logs

n/a

System and Environment Information

LifterLMS 4.19.0
WordPress 5.7.0
Theme: Twenty Twentyone

This issue has be recreated:

Browser, Device, and Operating System Information

n/a

Related User Information

Nobody reported it as far as I know.

thomasplevy commented 3 years ago

@eri-trabiccolo any idea if this is a regression/new bug introduced by #1555 or has it "always" (or for a while) worked like this and we just haven't noticed? I'm not sure it truly matters but I'm just curious if we should start with the new code from #1555 as a source of this problem or not.

eri-trabiccolo commented 3 years ago

@thomasplevy I don't think this is a regression. It's unrelated to me. It wasn't always so, it might be due to some change in wp.

thomasplevy commented 3 years ago

@eri-trabiccolo gotcha.

For now, I think let's let this hang with the "alternative" you've suggested and figure out a way to fix it in the future when we have capacity or requirements to do so.