mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #48193
[Bug 1766833] [NEW] maxlayoutrows is too low
Public bug reported:
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
** Affects: mahara
Importance: Undecided
Status: New
** Patch added: "mahara-maxlayoutrows-raise.patch"
https://bugs.launchpad.net/bugs/1766833/+attachment/5126944/+files/mahara-maxlayoutrows-raise.patch
--
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:
New
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
Follow ups
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Robert Lyon, 2018-10-25
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Nicolas Dunand, 2018-05-07
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Kristina Hoeppner, 2018-05-06
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Nicolas Dunand, 2018-05-04
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Robert Lyon, 2018-05-04
-
[Bug 1766833] A change has been merged
From: Mahara Bot, 2018-05-04
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Robert Lyon, 2018-05-04
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Steven, 2018-05-03
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Steven, 2018-05-03
-
[Bug 1766833] A patch has been submitted for review
From: Mahara Bot, 2018-04-26
-
[Bug 1766833] Re: maxlayoutrows is too low
From: Kristina Hoeppner, 2018-04-25