Problem/Motivation

The UI text on the Recent log messages page is rather long and conveluted and contains some general information that rather belongs on the Help page.

Proposed resolution

Update the UI text on the Recent log messages page of the Database Logging module, so that it's consistent with the hook_help text without duplicating it.

Remaining tasks

Update the UI text.

User interface changes

This is a UI text change.

API changes

None.

Data model changes

None.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Task because the additional help text on the administration module was too long.
Issue priority Minor
Unfrozen changes Unfrozen because it only changes strings.
CommentFileSizeAuthor
#6 2572577-6.patch943 bytesamfranco

Comments

ifrik created an issue. See original summary.

ifrik’s picture

ifrik
she/her
commented
Issue tags: +Usability
duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented
Issue tags: +Novice
pguillard’s picture

pguillard commented
Status: Active » Needs review

I suggest

The Database Logging module logs system events in the Drupal database.

Monitor your site or debug site problems on this page.

amfranco’s picture

amfranco commented

I am currently working on this issue at DrupalCon Barcelona 2015

amfranco’s picture

amfranco commented
Assigned: Unassigned » amfranco
StatusFileSize
new 2572577-6.patch943 bytes

I've changed the text based on #4 comment.

rumburak’s picture

rumburak commented

I checked the patch and it seems fine.

rumburak’s picture

rumburak commented
Status: Needs review » Reviewed & tested by the community
ifrik’s picture

ifrik
she/her
commented
Issue summary: View changes

Thanks for the propesed text and patch.
The text now concentrates on two issues: that the events are logged in the databbase (which is not evident from the page name), and it informs users that they can actually start debugging any problems from here.
The text can probably be shortened even further, but it's fine as it is.

Beta-evaluation added.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 6: 2572577-6.patch, failed testing.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs work » Reviewed & tested by the community

Random test failure.

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Getting this one in before strings freeze... definitely an improvement. :) https://twitter.com/YesCT/status/648062793103618048

Committed and pushed to 8.0.x. Thanks!

  • xjm committed 8138cd3 on 8.0.x 
    Issue #2572577 by amfranco, ifrik, pguillard: Update the UI texts for...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.