Moodle

Icon for database activity entries requiring approval needs to be made more visible

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9
  • Fix Version/s: 1.9.1
  • Component/s: Database activity module
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

The icon for database activity entries requiring approval was previously a big red tick. It's now a tiny black tick. Please could it be changed back.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

Well, the icon is everything but small:

http://cvs.moodle.org/moodle/pix/i/approve.gif?view=markup

was added (because was missing) as part of MDL-8188.

I guess it's more a thing of CSS or so (the size) ?

Ciao

Show
Eloy Lafuente (stronk7) added a comment - Well, the icon is everything but small: http://cvs.moodle.org/moodle/pix/i/approve.gif?view=markup was added (because was missing) as part of MDL-8188. I guess it's more a thing of CSS or so (the size) ? Ciao
Hide
Permalink
A. T. Wyatt added a comment -

No, actually I think the color is the important thing. As long as we cannot sort by recent changes, that color is very important. It is the only way we can find database entries that have been edited. If the check is black it is extremely hard to distinguish.

Show
A. T. Wyatt added a comment - No, actually I think the color is the important thing. As long as we cannot sort by recent changes, that color is very important. It is the only way we can find database entries that have been edited. If the check is black it is extremely hard to distinguish.
Hide
Permalink
Petr Škoda (skodak) added a comment -

icon is now bigger and you can sort by Approved
I hope the red is not important anymore

please test and reopen if needed, thanks for the report!

Show
Petr Škoda (skodak) added a comment - icon is now bigger and you can sort by Approved I hope the red is not important anymore please test and reopen if needed, thanks for the report!
Hide
Permalink
Helen Foster added a comment -

Thanks Petr I'll test this on Tuesday.

Show
Helen Foster added a comment - Thanks Petr I'll test this on Tuesday.
Hide
Permalink
Petr Škoda (skodak) added a comment -

reopening, going to make the icon bigger

Show
Petr Škoda (skodak) added a comment - reopening, going to make the icon bigger
Hide
Permalink
Petr Škoda (skodak) added a comment -

fixed in cvs - approve icon now bigger by default - size can be changed through CSS

Show
Petr Škoda (skodak) added a comment - fixed in cvs - approve icon now bigger by default - size can be changed through CSS
Hide
Permalink
Helen Foster added a comment -

Thanks Petr, the icon looks really good now

Show
Helen Foster added a comment - Thanks Petr, the icon looks really good now
Hide
Permalink
A. T. Wyatt added a comment -

Agree! I couldn't miss it! The size made up for not having red. And that was a terrific help. Thanks.

Show
A. T. Wyatt added a comment - Agree! I couldn't miss it! The size made up for not having red. And that was a terrific help. Thanks.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.