Clear log description needs clarification

my usability opinion would be:

1- a whole (dangerous) fieldset to hold a single button?
2- users allowed to access reports are also allowed to get ride of them?

R/1) provide two buttons, "Clear old messages" and "Clear ALL messages"
the first one, would be configurable for how long would be the "old" window frame, and that configuration might show up as part of the description, kind of
"Clear old message will permanently remove ... beyond the %time_window" making '%time_window' evaluate to "last half hour" / "3 hours and 15 minutes" / etc (I can provide the "format_minutes" function for this kind of messages)

R/2) before rendering/submitting the form check for user_access on 'administer site configuration' as well
meaning that for log clearance users will need both, to be able to "access" and also to "administer" log history

@#5

a) yes, I know, and I agree it shouldn't be there by default, maybe enabling it at admin/config/development/logging

b) when I want to be fast, I just want to get the last 10 top logs, or the last 1 hour logs, that is the job for a straightforward link, not a filter and then a clear button

c) nevertheless, a "Clear filtered" button would be very desirable

I was just saying that the core's fieldset is some... misplaced
it is also dangerous
and the most important, accessing the logs is not permission enough to be able to drop them

and the most important, accessing the logs is not permission enough to be able to drop them

+1

Also, when logs *are* cleared, a log entry should be generated that notes this action.

Looks like this will be D8, unless someone can bump it into D7.

Based on that and commends in #6 and #7, marking needs work...

#3: dblog_clear.patch queued for re-testing.

No screenshot to review

Re-rolled.

I am testing this ticket as part of the mentored sprint at Drupal Con 2016 New Orleans.

I tested the patch applied from #13 and was able to visually confirm the result.

I would like to comment that I think that there should be more visual separation between the text and the button in 8.2.x using the default admin theme. I could make an appropriate ticket in the Seven issue queue if such a thing exists.

I have tested this out and it appears to do what it is supposed to do, and have confirmed the result with the screenshot above.

Thanks @mikeohara. This will probably replaced with this patch that also needs a review if you want to check #2506449: Transform "Clear log messages" submit button into a link in admin/reports/dblog

Upon further review I discussed with @joelpittet providing emphases around the word "All". It seems like we do not use emphasis the same way anywhere else. It seems as though the convention is to use an <em>...</em> tag to show emphasis which is also more language agnostic. I have created a new patch as a result, it is attached as well as a screenshot of the new result.

@dagmar Ah, I didn't see this comment before I committed. I will take a look, thanks!

This is a duplicate of #2506449: Transform "Clear log messages" submit button into a link in admin/reports/dblog. Thanks to @dagmar for relating the two issues.