[MDL-65892] Inline reply field styling not consistent with other form fields - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 3.7
Fix Version/s: 3.7.1
Component/s: Accessibility, Forum, Themes
Labels:

Affected Branches:
MOODLE_37_STABLE
Fixed Branches:
MOODLE_37_STABLE
Pull from Repository:
git://github.com/mudrd8mz/moodle.git
Pull Master Branch:
~~MDL-65892~~-master-inlinereply
Pull Master Diff URL:
https://github.com/mudrd8mz/moodle/compare/f1a8db6911...MDL-65892-master-inlinereply
Testing Instructions:
Hide

Purge theme caches. This step could be skipped if you have a fresh installation.

sudo -u www-data php admin/cli/purge_caches.php --theme

Access to a course and use the inline reply feature in a forum.

CHECK that the reply textarea is styled consistently with standard BS4 form fields. That means it gets highlighted on focus, has rounded corners (depending on the browser) and inner padding between the border and the text.
Show
Purge theme caches. This step could be skipped if you have a fresh installation. sudo -u www-data php admin/cli/purge_caches.php --theme Access to a course and use the inline reply feature in a forum. CHECK that the reply textarea is styled consistently with standard BS4 form fields. That means it gets highlighted on focus, has rounded corners (depending on the browser) and inner padding between the border and the text.

Sprint:
Internationals - 3.8 Sprint 2

Description

As reported by mudrd8mz the inline reply field would look better with some space around the text (as illustrated in the attached screenshots). Additionally, when focused, the field does not get highlighted as other bootstrap based form fields.

Attachments

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

inline reply field with space around text.jpg
24 kB
11/Jun/19 10:49 PM
inline reply without space around text.png
26 kB
11/Jun/19 10:49 PM
MDL-65892_inlinereply.jpg
28 kB
03/Jul/19 9:43 AM
MDL-65892.jpg
28 kB
03/Jul/19 5:01 PM

Activity

People

Assignee:: David Mudrák (@mudrd8mz)

Reporter:: Helen Foster

Peer reviewer:: Mathew May

Integrator:: Sara Arjona (@sarjona)

Tester:: Anna Carissa Sadia

Participants:: Anna Carissa Sadia, CiBoT, David Mudrák (@mudrd8mz), Helen Foster, Mathew May, Sara Arjona (@sarjona)

Component watchers:: Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie, Huong Nguyen, Bas Brands, Mathew May

Votes:: 0 Vote for this issue

Watchers:: 5 Start watching this issue

Dates

Created:: 11/Jun/19 10:48 PM

Updated:: 04/Jul/19 11:58 PM

Resolved:: 04/Jul/19 11:58 PM

Fix Release Date:: 8/Jul/19

Time Tracking

Estimated:

Not Specified

Remaining:

Logged:

1h 11m