Skip to content
  • Lisa Seeto's avatar
    Bug 1827811: Move "Add comment" and "Details" into modal with "Comments" · be7af61f
    Lisa Seeto authored
    
    
    create modal using configureblock id, update various templates to use the modal for blocks:
    Image
    Note
    File(s) to download
    Journal/blogs
    add JS to open/close modal, add comments, delete comments, reply to comments, paginate comments.
    add JS to handle url that redirects to view/view.php with modal open and populated.
    Further changes to each block will be completed in child patches.
    Add segment headings to the modal. Fix display of comments to cover width of modal.
    Show comment removed msg in modal/page appropriately.
    Fix behat test to close modal before continuing.
    do not show existing comments if comments not allowed on artefact.
    add validation to modal for comments/attachments
    disable image as link when editing for image block
    trigger focus on modal for accessibility
    close modal on escape keydown event
    fix merge conflict in view/view.php
    fix merge conflict with master
    make configureblock modal play nicely when comment block is on the page
    update display of error messages to always show at top of modal so user doesn't have to scroll
    add accesibility to close modal on escape key down
    handle dock show/hide events for paginated pages in modal during validation
    create block-comments-details-header template file
    create details button and custom scss file with js to show and hide header
    update comment links to save add comment and details when commentcount == 0
    add active-block class to colour the block-header when active
    toggle the details-btn active css
    add _block-comments-details-header.scss to patch
    change class invisble to class d-none
    add block-comments-details-header.tpl to blocktypecontainerviewing.tpl
    code review changes
    add .no-modal class to links on group home page, dashboard and profile page.
    add block header to appear above the title
    change assessbtn from anchor tag to button to prevent scrolling.
    make all clickable links clickable on page load (ignore assess mode)
    fix js issues surrounding assessbtn move and show/hide blocks
    change cursor on dashboard, my profile page, group homepage to default when over disabled modal links
    update assessbtn to details btn
    create tooltip for hover over file download block titles
    fix containerElm console log error by not reattaching tinymce unneccessarily.
    fix null named attributes issues on e.target when span (icons) instead of anchor is clicked.
    
    Change-Id: I2e5218c040f8a9e252f785165ce9c566ecc2f56e
    Signed-off-by: default avatarLisa Seeto <lisaseeto@catalyst.net.nz>
    be7af61f