• 
      

    Add a JavaScript hook for extending the File Attachment thumbnail container UI.

    Review Request #12951 — Created April 19, 2023 and submitted

    Information

    Review Board
    release-6.x

    Reviewers

    This adds a JavaScript hook for extending the File Attachment thumbnail
    container UI. The hook can be used to display additional content on
    file attachment thumbnails and in the file attachment actions menu.

    This also adds a method to the File Attachment thumbnail view that allows
    callers to add an action to the file attachment actions menu.

    • Ran JS unit tests.
    • Ran unit tests.
    • Manually tested with the new "Download as PDF" action that's being added in
      in Power Pack.
    • Manually tested with a simple extension and view, the one in the example
      in the docs.
    • Built the docs and made sure they look ok.
    Summary ID
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    6f449b1cf964645cc794bcf7207e5da520f664b7
    Description From Last Updated

    Missing semicolon. Column: 11 Error code: W033

    reviewbotreviewbot

    Missing semicolon. Column: 12 Error code: W033

    reviewbotreviewbot

    Missing semicolon. Column: 34 Error code: W033

    reviewbotreviewbot

    line too long (87 > 79 characters) Column: 80 Error code: E501

    reviewbotreviewbot

    I think it would be nicer to use a template literal here instead of +

    daviddavid

    Let's call this $appendItem. It would also be nicer to use a template literal here instead of +

    daviddavid

    line too long (87 > 79 characters) Column: 80 Error code: E501

    reviewbotreviewbot

    '$appendItem' is defined but never used. Column: 15 Error code: W098

    reviewbotreviewbot

    line too long (87 > 79 characters) Column: 80 Error code: E501

    reviewbotreviewbot

    The ==== lines should be the same width as the name

    daviddavid

    Let's sort these alphabetically.

    daviddavid

    Sort alphabetically.

    daviddavid

    Sort alphabetically.

    daviddavid

    Sort alphabetically.

    daviddavid

    line too long (87 > 79 characters) Column: 80 Error code: E501

    reviewbotreviewbot

    line too long (87 > 79 characters) Column: 80 Error code: E501

    reviewbotreviewbot
    Checks run (2 failed)
    flake8 failed.
    JSHint failed.

    flake8

    JSHint

    maubin
    Review request changed
    Change Summary:

    Added some missing semicolons.

    Commits:
    Summary ID
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    125d1de81d3f009bd96ff33d557faec2d83e7ab1
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    76c0ca916355428e4bd5e055520acb99c835f664

    Checks run (1 failed, 1 succeeded)

    flake8 failed.
    JSHint passed.

    flake8

    david
    1. 
        
    2. Show all issues

      I think it would be nicer to use a template literal here instead of +

    3. Show all issues

      Let's call this $appendItem. It would also be nicer to use a template literal here instead of +

    4. 
        
    maubin
    Review request changed
    Commits:
    Summary ID
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    76c0ca916355428e4bd5e055520acb99c835f664
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    4f098d5199967e715297e5abe70bb0ac4891f2d6

    Checks run (2 failed)

    flake8 failed.
    JSHint failed.

    flake8

    JSHint

    maubin
    Review request changed
    Commits:
    Summary ID
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    4f098d5199967e715297e5abe70bb0ac4891f2d6
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    ed0688ed38ca41b6429c57ead4bafb86f587f4a3

    Checks run (1 failed, 1 succeeded)

    flake8 failed.
    JSHint passed.

    flake8

    david
    1. 
        
    2. Show all issues

      The ==== lines should be the same width as the name

    3. Show all issues

      Let's sort these alphabetically.

    4. Show all issues

      Sort alphabetically.

    5. Show all issues

      Sort alphabetically.

    6. Show all issues

      Sort alphabetically.

    7. 
        
    maubin
    Review request changed
    Commits:
    Summary ID
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    ed0688ed38ca41b6429c57ead4bafb86f587f4a3
    Add a JavaScript hook for extending the File Attachment thumbnail UI.
    6f449b1cf964645cc794bcf7207e5da520f664b7

    Checks run (1 failed, 1 succeeded)

    flake8 failed.
    JSHint passed.

    flake8

    david
    1. Ship It!
    2. 
        
    maubin
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-6.x (44e44e7)