1. 07 Oct, 2019 1 commit
  2. 05 Oct, 2019 1 commit
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Tidying up some lastminute problems · fa4085d5
      Cecilia Vela Gurovic authored
      like
      - allow add block button to drop only inside the grid
      - navigation block in collection to copy to other pages
      - Smart evidence annotation feedback forms init tinymce
      - showing the peer can see content message again
      - allow copying of page with old layout
      
      behatnotneeded
      
      Change-Id: I3f552ce8f89125e9fd233513c855879aa7678048
      fa4085d5
  3. 04 Oct, 2019 20 commits
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: checking old layout tables exists before using them · 279c2d49
      Cecilia Vela Gurovic authored
      - when deleting an institution
      - added a function to simulate the content of
      view_layout_column table that was deleted.
      This is needed when importing a portfolio with
      old layout
      - removed function addcustomlayout and some
      other code not used anymore
      - removed function get_layoutrows that was used
      when editing old layouts
      
      behatnotneeded
      
      Change-Id: Ibebf50103bf1811894e75a21a668c4700283e703
      279c2d49
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: removing layout tables from install · 24e838cf
      Cecilia Vela Gurovic authored
      - removing
      'usr_custom_layout'
      'view_layout_rows_columns'
      'view_layout'
      'view_layout_columns'
      
      behatbitneeded
      
      Change-Id: Ic53029dfc33b6366c38750d1453cbb64fd9c317e
      24e838cf
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Behat - block creation · 9cf3d8eb
      Cecilia Vela Gurovic authored
      - Data generator creates blocks for new gridstack layout
      - we don't look for nodes inside script elements
      
      behatnotneeded
      
      Change-Id: I0e791c431ab6b80e75ae397290a20b3ec9fba836
      9cf3d8eb
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Blocks display fixes · ccc2446d
      Cecilia Vela Gurovic authored
      - assessment block: tinymce and the asseesment js
      need to be initialized after the block is loaded
      - signoff block: sing-off modal is showing now
      - embedded media init functions need to be loaded after
      the blocks are added to the grid
      - wall post block initializes the tinyMCE after it's loaded
      and refreshes the content of the block after a new post is made
      - comment block: update block size when adding a comment
      - scrolling to the block when a new one is added
      - modified the text for button to add a new block to the page
        'Drag to add a new block' for normal mode
        'Click to add a new block' for accessibility mode
      - add a block to the bottom of the page by default so it won't
      break the layout
      - add SE annotation to the bottom of the page
      - added 'Add block' button event handlers after trnaslating a view
      - annotation feedback loads the tinyMCE
      
      behatnotneeded
      
      Change-Id: I7642cacde729a2a85b497e077452de792ec72eb7
      ccc2446d
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Accessibility settings and layout · 25082988
      Cecilia Vela Gurovic authored
      - Added an option to set the profile to accessible.
      This can be set in account preferences and
      enables one extra field in the settings
      of a view, in the basic settings section, to make
      the layout accessible.
      
      - by default, if the user account is accessible then
      the pages will be crated as accessible.
      To have a non accessible page, the option un view settings
      needs to be set as 'No'
      
      - Accessible layout only allows to add blocks with width=12
      that is the same with of the grid.
      This makes the blocks show as a sequence one on top of the other
      like a list. The user can only reorder the position
      they have in the list.
      
      - When a page is accessible, the adding a block
      by drag & drop is disabled
      
      - Reordering of blocks is accessible by keyborad
      
      - The 'add block' button is reachable by keyboard
      
      - add an intro to the page for the screen reader
      to explain how the page works
      
      - add identification to each block for screen reader,
      blocktype and title (if it’s set)
      
      TODO
      * When adding a block to the bottom or the top of the page,
      the page should be scrolled to that block
      
      * Floating menu: should have a way to be recognized as a menu
      by the screen reader. Check the access keys for the menu.
      
      behatnotneeded
      
      Change-Id: I08417f0f11d747a67900c88c2f675ef5f85b7499
      25082988
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Importing views with grid layout · af08a138
      Cecilia Vela Gurovic authored
      Importing views and collection with new grid layout
      
      - leap2A files containing views/collections with
      new grid layout are imported and created with the new layout
      
      - leap2A files containing collections/view with old layout
      will be created with a grid layout. This new layout is
      the translation of the old layout they had in the leap2A file.
      The heights of the blocks will be updated when the
      views are first displayed in the browser.
      
      behatnotneeded
      
      Change-Id: Ia43b7a5c11109023d3caa65ba16e3b6aebcdbe75
      af08a138
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Exporting views with grid layout · b6cd876f
      Cecilia Vela Gurovic authored
      In exporting section:
      /export/index.php
      
      - Show preview of page both for old layout pages and
      new grid layout
      - Export views and collections with new grid layout
      in html
      - Export views and collections with new grid layout
      in leap2A
      - views with old layout are exported as before
      
      NOTE
      borrow some code from https://reviews.mahara.org/#/c/10157/
      as an early rebase (the patch is not on master yet)
      
      behatnotneeded
      
      Change-Id: I0c36c110c6b9ef7ae3edc4d0662a0dbab302684a
      b6cd876f
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Make view versioning work with gridstack · 7b17b833
      Cecilia Vela Gurovic authored
      - Only allow to create a new version if the view
      is using new gridstack layout.
      
      In timeline version section:
      - For view versions that were created with old layout,
      show them as they are
      (we dont translate old view versions to new layout)
      - For view versions that are created with new grid layout,
      they are also displayed in the timeline view with the new grid
      layout
      
      behatnotneeded
      
      Change-Id: I1abdae9e5e547dcc9bce97cac127085786dcac8a
      7b17b833
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Copying a page (in old or new layout) creates a page with new layout · f342e41b
      Cecilia Vela Gurovic authored
      - Copy a page when original view is using
      grid layout
      - Copy a page when original view is using
      old layout: Translates the original page layout
      without modifying it,
      and creates a new page with new grid layout
      The blocks in the new page will not have height
      assigned. The height of the blocks is updated
      on first load of the page in the browser.
      This can happen when editing the page or displaying it.
      - Copy a collection with old layout: creates the new collection
      with new grid layout pages. Each page needs to be loaded on the
      browser to update the block heights
      
      behatnotneeded
      
      Change-Id: I43b5f9050be192c057cb1803dcd69575c137a9cc
      f342e41b
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Translate old views to new gridstack layout on edit · 32de0b18
      Cecilia Vela Gurovic authored
      Old layout pages will be displayed as they were
      with the old layout.
      
      New pages are created with the new grid layout.
      
      If we try to edit a page with an old layout,
      the page will be translated into the new grid layout.
      
      To translate from an old layout to a new grid layout,
      we respect the row, column and order of blocks inside each cell.
      
      All blocks in the same row in old layout will be together
      in the new layout, and the first block of each column
      of the same row are displayed at the same level to each other
      in the new layout.
      
      The blocks will respect the column order
      they had in the old layout.
      
      A column of the old layout can have many blocks. In the
      new grid layout they respect the same order they had.
      
      As we have 12 columns in the new grid layout,
      the blocks that where included in a 5 column row,
      will be displayed in smaller columns in the new layout.
      
      TODO:
      confirmation before trnaslating a page to the new grid layout
      
      behatnotneeded
      
      Change-Id: I46f5cdbacb026519631503437c10d33f1b2800e5
      32de0b18
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Making magic block work with gridstack layout · 66c08ce0
      Cecilia Vela Gurovic authored
      In view edit mode, dragging the button '+' on the
      right top menu, will add a block in the page
      with dimensions height=3 and width=4
      
      behatnotneeded
      
      Change-Id: I5f4334886f6685bd047cb5bfbf0d59dd3880109a
      66c08ce0
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: create a block- fixing the saving of a new block · 0772a240
      Cecilia Vela Gurovic authored
      behatnotneeded
      
      Change-Id: Ib20121937f6b7c64df65a620acbca93dd683d18f
      0772a240
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: block resizing to fit content · 565776fb
      Cecilia Vela Gurovic authored
      should not resize blocks in view mode
      
      in view mode, if the content of the block is static,
      then the block dimensions should not change but the
      content of the block should be made smaller
      
      expand the height of a block only if the content is dynamic,
      like the 'Journal' block
      
      behatnotneeded
      
      Change-Id: Ie8183736580e7e592b24e36f4a16618b96a5bf4e
      565776fb
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: remove layout button from navigation · f9295a81
      Cecilia Vela Gurovic authored
      + Also remove not used templates. They should have been deleted
      in commit 84119067
      that unifies layout and settings section
      
      behatnotneeded
      
      Change-Id: Icd90bd4980b9be566f7e20aee6271476edafe321
      f9295a81
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Display a page in view mode · c0f57e7d
      Cecilia Vela Gurovic authored
      The blocks saved with the new layout
      are now displayed in page edit and view mode
      
      behatnotneeded
      
      Change-Id: Ic8c1011745593be552cc1a4377a7351cc9a04bb9
      c0f57e7d
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Creating a new page with gristack layout · 2c861ec1
      Cecilia Vela Gurovic authored
      add a new block, place it anywhere in the grid, edit configuration
      and delete it.
      
      It will save it in the db on table block_instance_dimension
      but not display it in view or edit mode yet
      
      Also included a small fix in xmldb postgres class generator
      to handle reserved words in getAlterFieldSQL function
      
      Failing tests:
      
      - most of them are failing when adding a new block to the page
      because they expect to have a modal to choose the position
      where to place block. That modal was removed for this patch but
      there will be a similar one in patch:
       https://reviews.mahara.org/#/c/9952
      and those tests will be fixed there
      
      - a few tests failed when they couldn't find text inside the
      blocks because they are not expanded to fit the content yet.
      This is added in patch:
       https://reviews.mahara.org/#/c/9986
      and they will be fixed there
      
      behatnotneeded
      
      Change-Id: If4521a6315f6e8cc5d88693f536946dace359288
      2c861ec1
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: removing helper functions to reorder cells in old layout · 2a9211b3
      Cecilia Vela Gurovic authored
      they won't be needed anymore
      
      also remove use of properties
      private $dirtyrows;
      private $dirtycolumns;
      
      Change-Id: I31e0c8296e72e9dd6b2381170864bdc52af766b7
      2a9211b3
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Remove 'addcolumn' functionality when editing a view · 5f57b602
      Cecilia Vela Gurovic authored
      behatnotneeded
      
      Change-Id: I99b248e03ffe5b23e622c46ce14765bdf1e47195
      5f57b602
    • Cecilia Vela Gurovic's avatar
      Bug 1813987: Remove edit layout section · d45aed8d
      Cecilia Vela Gurovic authored
      - also remove the layout pieform
      and the custom layout javascript code
      
      behatnotneeded
      
      Change-Id: I3e2e2b3f09106dd2a341a40d178feca693194e64
      d45aed8d
    • Lisa Seeto's avatar
      Bug 1844470: Move "Display page" and "Return to pages and collections" · 466b4a8a
      Lisa Seeto authored
      to right-hand side on "Edit" screen
      
      - create new edit page action buttons for return to <location> button
      - delete references to the old display page link created in file editviewtabs.tpl
      - delete file editviewtabs.tpl
      - change the edit button in editviewpageactions.tpl into a display button
      - change behat tests to reflect the change from Edit to Display page
      - change the Behat instruction for And I display the page to target updated element
      - delete the old display button styling from the style guide
      - move url, title display logic into new func get_return_to_url_and_title in lib/view.php
      - adjust blocks.php, access.php, editlayout.php to assign url, title using new func
      
      behatnotneeded
      
      Change-Id: Iedf2882c42fa1d204e438a23842834fbb9a709f9
      Signed-off-by: default avatarLisa Seeto <lisaseeto@catalyst.net.nz>
      466b4a8a
  4. 01 Oct, 2019 1 commit
    • Alexander Del Ponte's avatar
      Bug 1837999: Extend the plans functionality · 6a10a44d
      Alexander Del Ponte authored
      The plans are extended to allow for the creation
      of template plans in the personal portfolio area.
      These templates can then be used to create plans
      in many groups for students to copy into their own
      portfolio area and complete as assignments.
      
      behatnotneeded: To come later
      
      Change-Id: Ifbdf70541c0a0a254af3cbb9390d883a60e591d0
      6a10a44d
  5. 20 Sep, 2019 1 commit
  6. 18 Sep, 2019 1 commit
  7. 10 Aug, 2019 1 commit
    • Shalu Garg's avatar
      Bug 1839499: Site pages and collections, Site unavailable error after enabling lti plugin · 1d2bad4e
      Shalu Garg authored
      Fixing the issue of sql exception on pages & collections page after logging in as a non admin
      user on enabling lti_assessment plugin. This is caused by calling a select statement for
      lti_assessmentid column with aggregate function and group by condition,
      but missing group by condition for lti_assessmentid.
      
      Change-Id: I0a040018fadcf543452b24174964e929f286e552
      1d2bad4e
  8. 06 Aug, 2019 1 commit
  9. 04 Aug, 2019 2 commits
    • Robert Lyon's avatar
      Bug 1829940: Restrict the chooser to one 'add' button · 4d9a5645
      Robert Lyon authored
      Done in this patch:
      - restricted the list to one shortcut button (the placeholder one)
      - cleaned up old code where one could edit the block on the without
      modal
      - moved the button to be above content layout and next to the
      instructions section
      - move the block help link to within placeholder block and be an 'i'
      on 'Content types' heading
      
      behatnotneeded
      
      Change-Id: I8a85842ffef5e50c62aca73a4b33c22579b160ca
      Signed-off-by: Robert Lyon's avatarRobert Lyon <robertl@catalyst.net.nz>
      4d9a5645
    • Robert Lyon's avatar
      Bug 1829940: Placeholder option update page with block of choice · 882259ee
      Robert Lyon authored
      This patch does the following:
      - Make 'placeholder' configure options clickable so that on clicking
      an option updates the block on the page and changes the configure form
      to the form of the blocktype chosen
      - On 'add placeholder' -> 'choose option' we have 'remove' option on
      form and it removes the block from the page
      - On 'add placeholder' -> 'save' -> 'configure block' -> 'choose option'
      we have 'cancel' option on form and it reverts the block back to
      'placeholder'
      
      behatnotneeded
      
      Change-Id: I1681ef92c5ccc14cda2167a040bb751501bfdbcd
      Signed-off-by: Robert Lyon's avatarRobert Lyon <robertl@catalyst.net.nz>
      882259ee
  10. 18 Jul, 2019 1 commit
  11. 24 Jun, 2019 1 commit
  12. 23 Jun, 2019 1 commit
  13. 16 Jun, 2019 1 commit
  14. 26 May, 2019 1 commit
    • Kevin Dibble's avatar
      Bug 1044878: Editing duplicated content · 62d8203a
      Kevin Dibble authored
      Added a new feature that the user has to choose yes to
      start editing copy. This will force users to interact with the
      form before editing with the choice that they are editing
      all copies of this content.
      
      When selecting to use content from another note, the
      form will update based on their choice. The editor will
      remain open if yes has been selected. The editor will
      be hidden and only the content will show if 'Yes' has
      not be selected.
      
      This is a UI element only - no decision is saved to the
      DB as this is an editor based decision.
      
      behatnotneeded
      
      Change-Id: If44522c00672b296242893f0c876e7ff37561dd2
      62d8203a
  15. 13 May, 2019 1 commit
  16. 30 Apr, 2019 1 commit
  17. 12 Apr, 2019 1 commit
  18. 30 Mar, 2019 1 commit
    • Gregor Anzelj's avatar
      Bug 717196: Isolated institutions · 258c4744
      Gregor Anzelj authored
      Isolated institutions is a feature that allows locking down
      access for members of institutions so that they are separated
      entirely and disallow contact between members of one
      institution with members of another institution.
      
      behatnotneeded
      
      Change-Id: Ib94681aff5543ad26887f24b2eb7fce9d0c8b8d2
      258c4744
  19. 28 Mar, 2019 1 commit
  20. 14 Mar, 2019 1 commit