[MDL-75430] "Width in list view" is ignored for the picture field type of the Database activity - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 4.1
Fix Version/s: 4.1
Component/s: Database activity module
Labels:
- ci
- hqteam_moppies
- regression
- triaged

Affected Branches:
MOODLE_401_STABLE
Fixed Branches:
MOODLE_401_STABLE
Epic Link:
Moodle 4.1 Database project
Pull from Repository:
https://github.com/raortegar/moodle.git
Pull Master Branch:
~~MDL-75430~~-master
Pull Master Diff URL:
https://github.com/raortegar/moodle/compare/35b993b694...MDL-75430-master
Testing Instructions:

Hide

Manual test

It would be nice to follow the manual testing instructions to confirm the size of the picture for the list view is correct

1- Go to any course with an admin user and activate the "edit mode"
2- Create a database activity (Add the name and leave the rest of the fields as default)
3- Go to the tab "Fields" of the database and add a new field of type "Picture"
a) Add "Filed name" : "test"
b) Add "Width in single view": 500
c) Add "Width in list view": 50
d) Click on the button "Add"

4- Go to "Templates" tab and save the template as default
5- Go to "Database" tab, and "add entry"
a) Upload a picture
b) Save

6) The single view will be display as default, select "List view"

Result:
The Test picture added before will be reseized to 50px

Show
Manual test It would be nice to follow the manual testing instructions to confirm the size of the picture for the list view is correct 1- Go to any course with an admin user and activate the "edit mode" 2- Create a database activity (Add the name and leave the rest of the fields as default) 3- Go to the tab "Fields" of the database and add a new field of type "Picture" a) Add "Filed name" : "test" b) Add "Width in single view": 500 c) Add "Width in list view": 50 d) Click on the button "Add" 4- Go to "Templates" tab and save the template as default 5- Go to "Database" tab, and "add entry" a) Upload a picture b) Save 6) The single view will be display as default, select "List view" Result: The Test picture added before will be reseized to 50px

Story Points:
1
Sprint:
HQ Sprint 0.2 Database

Description

Steps to reproduce it:

Create a database activity.
Add a Picture field and set:
- Width in single view to 300
- Width in list view to 50
Go to templates (to generate the templates).
Add an entry with an image.
Go to the List view.

Expected behaviour: the image is smaller in the list view than in the single view.

Current behaviour: the image is always displayed with the size defined for the single view.

In 4.0 is working as expected, so it seems a regression from the database project.

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

MDL-75430_master.png
49 kB
07/Sep/22 9:32 AM
Screenshot from 2022-08-11 19-35-12.png
52 kB
12/Aug/22 1:43 AM

Issue Links

Discovered while testing

MDL-75252 Improve Image gallery preset templates

Closed

Activity

People

Assignee:: Raquel Ortega

Reporter:: Sara Arjona (@sarjona)

Peer reviewer:: Laurent David

Integrator:: Ferran Recio

Tester:: John Edward Pedregosa

Participants:: CiBoT, Ferran Recio, John Edward Pedregosa, Jun Pataleta, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)

Component watchers:: Amaia Anabitarte, Carlos Escobedo, Laurent David, Sabina Abellan, Sara Arjona (@sarjona)

Votes:: 0 Vote for this issue

Watchers:: 6 Start watching this issue

Dates

Created:: 10/Aug/22 11:23 PM

Updated:: 09/Sep/22 10:49 AM

Resolved:: 09/Sep/22 10:49 AM

Fix Release Date:: 28/Nov/22

Time Tracking

Estimated:

Remaining:

Logged:

5h 4m