TexasDigitalLibrary / Vireo

Vireo is a turnkey Electronic Thesis and Dissertation (ETD) Management System.
https://texasdigitallibrary.atlassian.net/wiki/spaces/VUG/pages/87490642/About
GNU General Public License v2.0
44 stars 33 forks source link

Issue 1858: Set wysiwyg parameter on all lockingtextarea fields. #1859

Closed kaladay closed 11 months ago

kaladay commented 11 months ago

Resolves #1858

This applies the same fix from PR #1792 (commit ef5004bd7b6a4ec5c37a159e9693893a722aaa6f).

The wysiwyg attribute needs to be specified now on the lockingtextarea field.

ghost commented 11 months ago

Perhaps ng-switch in the template can provide a default.

https://github.com/TexasDigitalLibrary/Vireo/blob/4.2.4_staging/src/main/webapp/app/views/directives/lockingTextArea.html

Has this been tried to not require wysiyg being specified?

<div ng-switch="wysiwyg">
  <div ng-switch-when="true">
    <textarea ui-tinymce="tinymceOptions" ng-model="$parent.scopeValue">
      </textarea>
  </div>
  <div ng-switch-default>
    <textarea class="form-control locking-text-area-not-wysiwyg" ng-model="$parent.scopeValue"
      ng-keypress="$parent.nonWysiwygTyping($event)" ng-blur="$parent.nonWysiwygBlur()">
      </textarea>
  </div>
</div>