Launchpad: https://bugs.launchpad.net/mahara/+bug/1766833 Gerrit: https://reviews.mahara.org/#/c/8813/ Bug description: increase maxlayoutrows -------------------------------------------------------------- Environment tested: Master Browser tested: Chrome
=================== Acceptance Criteria: 1. Portfolio / Group page "Layout" - max rows = 20 rows 2. Shows smaller preview rows for layouts having >= 60 rows 3. Page displays correctly when copied by other users Page displays correctly when exported to Standalone HTML website Page displays correctly when exported to Leap2A =================== Manual Test Script =================== Preconditions: 1. "Group 101" exists (Admin group) and has the following a. page is SHARED WITH = public b. Advanced options > Allow copying = yes 2. "User1" exists and has the following a. Belongs to "A1 Institution" b. Is owner of "Group 101" 3. "User1" exists and has the following a. Belongs to "A1 Institution" b. Is owner of "Group 101" Test Script: 1. Log in as User1 2. Browse to Main menu > Portfolio > Pages and collections > 3. Add a page - Name = "max 20 layout rows (new)" 4. open "Layout" accordion tab 5. Scroll to "Create custom layout" (bottom of page) 6. Add as many single column rows as is possible 7. Confirm that the maximum rows allowed is 20 rows ✔ 8. Click the create new layout button to create you 20 row custom layout 9. Confirm that an icon depicting the 20 row custom layout is displayed and has a radio button that can be selected ✔ 10. Select the radio button on the 20 row custom layout icon and click the save button 11. Confirm that the Newly created page has 20 rows available ✔ 12. Add a text block to each of the rows 13. Share the page with public and allow copying 14. Create a Page for Group 101 and follow the above steps 15. Confirm that the page displays as expected a. Confirm page displays text blocks in each of the 20 rows ✔ 16. Login as User 2 17. Browse to Admin's portfolio page "max layout rows 20 (new)" 18. Copy the page and rename it to "User1 max layout rows 20 (new)" 19. Confirm that the page displays as expected a. Confirm page displays text blocks in each of the 20 rows ✔ 20. Export the page to a Standalone HTML website 21. Confirm the html page displays text blocks in each of the 20 rows ✔ 22. Export the page to Leap2A file 23. Import the file into User3 portfolio 24. Confirm that when the imported page displays text blocks in each of the 20 rows ✔ 25. Confirm the settings for the imported page are the same as the original ✔ Catalyst QA Approved ✔ -- You received this bug notification because you are a member of Mahara Contributors, which is subscribed to Mahara. Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it! https://bugs.launchpad.net/bugs/1766833 Title: maxlayoutrows is too low Status in Mahara: In Progress Bug description: The maximum number of rows for a view is arbitrarily set to 6 in lib/view.php ($maxlayoutrows), which may be too small for some uses. The code also references some file (layoutpreview.php) which is not present (as an explanation for this limitation ?). Discussion on https://mahara.org/interaction/forum/topic.php?id=6413 has Robert Lyon and Aaron Wells stating (back in 2014-2015) that this could be changed. Therefore I propose a small modification, which : 1. raises this limit to 20 2. shows smaller preview rows for layouts having >= 60 rows Attached patch for Mahara 17.x-18.04 To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/1766833/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : mahara-contributors@lists.launchpad.net Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp