openproject/frontend/app/components/work-packages/work-package-comment/work-package-comment.direct...

<div id="work-package-activites-container">
  <div id="work-package-comment-container">

    <ng-transclude ng-if="!vm.showAbove"></ng-transclude>
    <div class="work-package--details--long-field work-packages--activity--add-comment" ng-if="vm.canAddComment">
      <div class="wp-edit-field inplace-edit" ng-switch="vm.active">
        <form ng-switch-when="true"
              name="wp-edit-form-coment"
              ng-submit="vm.handleUserSubmit()"
              role="form"
              tabindex="-1">

          <label for="{{::vm.htmlId}}"
                 class="hidden-for-sighted">
            {{::vm.text.addComment}}
          </label>

          <ng-include src="vm.field.template"></ng-include>
        </form>
        <div class="inplace-edit--read" ng-switch-when="false">
          <accessible-by-keyboard
            class="inplace-editing--trigger-container"
            span-class="inplace-editing--container"
            link-class="inplace-editing--trigger-link"
            aria-label="{{ ::vm.text.editTitle }}"
            execute="vm.activate()"
            focus="vm.shouldFocus()">
            <span class="inplace-edit--read-value -default">
              <span ng-bind="::vm.text.placeholder"></span>
            </span>
            <span class="inplace-edit--icon-wrapper">
              <icon-wrapper icon-name="edit" icon-title="{{ ::vm.text.editTitle }}"></icon-wrapper>
            </span>
          </accessible-by-keyboard>
        </div>
      </div>
  </div>
  <ng-transclude ng-if="vm.showAbove"></ng-transclude>
</div>
Replace exclusive-edit with transcluded comment directive This commit removes the somewhat superfluous exlusiveEditController in favor of event-based passing of quoted text to the correct comment field. The comment directive now decides for itself whether it wants to be placed above or below the activities and uses transclusion for that purpose. Editing comments is now no longer exlusive, but that shouldn't be necessary anyway. 9 years ago			`<div id="work-package-activites-container">`
Fix activity spec 8 years ago			`<div id="work-package-comment-container">`
Move WP comment directive partial code to directive markup 9 years ago
Rewrite comment directive in TypeScript without inplace 8 years ago			`<ng-transclude ng-if="!vm.showAbove"></ng-transclude>`
Add styles for textareas (comment, description) 8 years ago			`<div class="work-package--details--long-field work-packages--activity--add-comment" ng-if="vm.canAddComment">`
Remove angular-busy 8 years ago			`<div class="wp-edit-field inplace-edit" ng-switch="vm.active">`
Rewrite comment directive in TypeScript without inplace 8 years ago			`<form ng-switch-when="true"`
			`name="wp-edit-form-coment"`
[23896] Submit and close edit mode on enter for most edit fields When editing a single field (except select, checkbox), pressing enter triggers the `submit` handler of the surrounding form. This form submit is disabled when in edit mode, since the wp-edit-field uses that submit also for saving on blur, which we deliberately disable in this case. We can instead capture another event on those fields where it makes sense that submits the field after pressing enter, but only IN edit mode (since without it, submit handles it). https://community.openproject.com/work_packages/23896 8 years ago			`ng-submit="vm.handleUserSubmit()"`
Rewrite comment directive in TypeScript without inplace 8 years ago			`role="form"`
			`tabindex="-1">`

			`<label for="{{::vm.htmlId}}"`
			`class="hidden-for-sighted">`
			`{{::vm.text.addComment}}`
			`</label>`

			`<ng-include src="vm.field.template"></ng-include>`
			`</form>`
			`<div class="inplace-edit--read" ng-switch-when="false">`
Move WP comment directive partial code to directive markup 9 years ago			`<accessible-by-keyboard`
Rewrite comment directive in TypeScript without inplace 8 years ago			`class="inplace-editing--trigger-container"`
			`span-class="inplace-editing--container"`
			`link-class="inplace-editing--trigger-link"`
Fix comment accessibility 1. Make the title a label (we won't need a title for seeing users anyway, the full text is there). 2. Remove invalid `\|\| undefined` in the ng-disabled check 8 years ago			`aria-label="{{ ::vm.text.editTitle }}"`
Include review feedback 8 years ago			`execute="vm.activate()"`
			`focus="vm.shouldFocus()">`
Restore italic default styling of comment field 8 years ago			`<span class="inplace-edit--read-value -default">`
Rewrite comment directive in TypeScript without inplace 8 years ago			`<span ng-bind="::vm.text.placeholder"></span>`
			`</span>`
			`<span class="inplace-edit--icon-wrapper">`
			`<icon-wrapper icon-name="edit" icon-title="{{ ::vm.text.editTitle }}"></icon-wrapper>`
			`</span>`
Move WP comment directive partial code to directive markup 9 years ago			`</accessible-by-keyboard>`
			`</div>`
Rewrite comment directive in TypeScript without inplace 8 years ago			`</div>`
Move WP comment directive partial code to directive markup 9 years ago			`</div>`
Restore italic default styling of comment field 8 years ago			`<ng-transclude ng-if="vm.showAbove"></ng-transclude>`
Replace exclusive-edit with transcluded comment directive This commit removes the somewhat superfluous exlusiveEditController in favor of event-based passing of quoted text to the correct comment field. The comment directive now decides for itself whether it wants to be placed above or below the activities and uses transclusion for that purpose. Editing comments is now no longer exlusive, but that shouldn't be necessary anyway. 9 years ago			`</div>`