Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Activity Log: Tasklist restict the task list to 3 items on page load #26710

Merged
merged 1 commit into from
Aug 15, 2018

Conversation

enejb
Copy link
Member

@enejb enejb commented Aug 15, 2018

Implements #25707

Currently when you are on the activity log and your site shows you all the update items.
Which can be quite overwhelming.

This PR reduced them to just 3 and shows a expand Show more link. No link is shown if we have less then or equal to 3 items to update.

Before:
screen shot 2018-08-15 at 10 50 00 am

After:
screen shot 2018-08-15 at 10 49 54 am

Mobile:
screen shot 2018-08-15 at 10 50 09 am

To test:
Check that there are no console log errors in your dev console.
Set a site that has more then 3 items to update visit it and note that everything looks as expected.
Click on the show more link and it will show you all the items that need updating.

Note if you have less then 3 items to update that all items are shown.

@enejb enejb added [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR [Feature] Activity Log labels Aug 15, 2018
@enejb enejb self-assigned this Aug 15, 2018
@matticbot
Copy link
Contributor

} = this.props;
const { translate, name, version, type, disable, linked, slug, siteSlug } = this.props;

let updateType = translate( 'Plugin update available' );
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! I've found a possible matching string that has already been translated 19 times:
translate( 'Plugin update available', 'Plugin updates available' ) ES Score: 16
See 1 additional suggestion in the PR translation status page

Help me improve these suggestions: react with 👎 if the suggestion doesn't make any sense, or with 👍 if it's a particularly good one (even if not implemented).


let updateType = translate( 'Plugin update available' );
if ( 'theme' === type ) {
updateType = translate( 'Theme update available' );
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! I've found a possible matching string that has already been translated 21 times:
translate( 'Theme update available', 'Theme updates available' ) ES Score: 13

Help me improve these suggestions: react with 👎 if the suggestion doesn't make any sense, or with 👍 if it's a particularly good one (even if not implemented).

@eliorivero
Copy link
Contributor

Functionality-wise, this works fine 🐑

captura de pantalla 2018-08-15 a la s 18 21 31

captura de pantalla 2018-08-15 a la s 18 24 55

However, it seems there was a regression at some point since the line height of the items has changed and they're cramped now, and the color of the plugin name is different.
As a reference, here's how updates looked previously, taken from last update I made to them in #25765 and approved by @keoshi

@enejb enejb merged commit 55c0879 into master Aug 15, 2018
@enejb enejb deleted the update/activity-log-updates-listing branch August 15, 2018 21:39
@matticbot matticbot removed [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Aug 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants