Skip to content

Commit 650d116

Browse files
authored
Merge pull request #527 from ProgressPlanner/develop
v1.6.1
2 parents cf68476 + 03d743a commit 650d116

File tree

7 files changed

+61
-44
lines changed

7 files changed

+61
-44
lines changed

CHANGELOG.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
= 1.6.1 =
2+
3+
Bugs we fixed:
4+
5+
* "Collect your point" not closing the "Test if your website can send emails correctly" popover.
6+
* Snooze task tooltip overaly not closing properly.
7+
18
= 1.6.0 =
29

310
Enhancements:

assets/js/suggested-task.js

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -436,37 +436,6 @@ prplSuggestedTask = {
436436
}
437437

438438
switch ( action ) {
439-
case 'snooze':
440-
tooltipActions
441-
.querySelector( elClass )
442-
?.setAttribute( 'data-tooltip-visible', 'true' );
443-
break;
444-
445-
case 'close-snooze':
446-
// Close the radio group.
447-
tooltipActions
448-
.querySelector(
449-
`${ elClass }.prpl-toggle-radio-group-open`
450-
)
451-
?.classList.remove( 'prpl-toggle-radio-group-open' );
452-
// Close the tooltip.
453-
tooltipActions
454-
.querySelector( `${ elClass }[data-tooltip-visible]` )
455-
?.removeAttribute( 'data-tooltip-visible' );
456-
break;
457-
458-
case 'info':
459-
tooltipActions
460-
.querySelector( elClass )
461-
?.setAttribute( 'data-tooltip-visible', 'true' );
462-
break;
463-
464-
case 'close-info':
465-
tooltipActions
466-
.querySelector( elClass )
467-
.removeAttribute( 'data-tooltip-visible' );
468-
break;
469-
470439
case 'move-up':
471440
case 'move-down':
472441
if ( 'move-up' === action && item.previousElementSibling ) {

assets/js/web-components/prpl-interactive-task.js

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/* global HTMLElement */
1+
/* global HTMLElement, prplSuggestedTask */
22

33
/**
44
* Register the custom web component.
@@ -73,17 +73,22 @@ class PrplInteractiveTask extends HTMLElement {
7373
*/
7474
completeTask() {
7575
const providerId = this.getAttribute( 'provider-id' );
76-
const components = document.querySelectorAll( 'prpl-suggested-task' );
76+
const tasks = document.querySelectorAll(
77+
'#prpl-suggested-tasks-list .prpl-suggested-task'
78+
);
7779

78-
components.forEach( ( component ) => {
79-
const liElement = component.querySelector( 'li' );
80-
if ( liElement.dataset.taskId === providerId ) {
80+
tasks.forEach( ( taskElement ) => {
81+
if ( taskElement.dataset.taskId === providerId ) {
8182
// Close popover.
8283
document
8384
.getElementById( 'prpl-popover-' + providerId )
8485
.hidePopover();
85-
// Complete task.
86-
component.runTaskAction( liElement.dataset.taskId, 'complete' );
86+
87+
const postId = parseInt( taskElement.dataset.postId );
88+
89+
if ( postId ) {
90+
prplSuggestedTask.maybeComplete( postId );
91+
}
8792
}
8893
} );
8994
}
Lines changed: 32 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,19 +12,49 @@
1212
*
1313
* @package Progress_Planner
1414
*/
15-
class Update_150 {
15+
class Update_161 {
1616

17-
const VERSION = '1.5.0';
17+
const VERSION = '1.6.1';
1818

1919
/**
2020
* Run the update.
2121
*
2222
* @return void
2323
*/
2424
public function run() {
25+
// Migrate the badges.
26+
$this->migrate_badges();
27+
28+
// Migrate the tasks.
2529
$this->migrate_tasks();
2630
}
2731

32+
/**
33+
* Migrate the badges.
34+
*
35+
* @return void
36+
*/
37+
private function migrate_badges() {
38+
// Get all badges.
39+
$badges = \progress_planner()->get_settings()->get( 'badges', [] );
40+
41+
foreach ( $badges as $badge_id => $badge ) {
42+
43+
// We are only migrating monthly badges.
44+
if ( 0 !== strpos( $badge_id, 'monthly-' ) ) {
45+
continue;
46+
}
47+
48+
if ( ! isset( $badges[ $badge_id ]['points'] ) ) {
49+
// We are just adding the points to the badge, for the new data structure - 10 is the max points for a badge.
50+
$badges[ $badge_id ]['points'] = 10 - (int) $badge['remaining'];
51+
}
52+
}
53+
54+
// Set the badges.
55+
\progress_planner()->get_settings()->set( 'badges', $badges );
56+
}
57+
2858
/**
2959
* Migrate the tasks.
3060
*

progress-planner.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
* Description: A plugin to help you fight procrastination and get things done.
1010
* Requires at least: 6.6
1111
* Requires PHP: 7.4
12-
* Version: 1.6.0
12+
* Version: 1.6.1
1313
* Author: Team Emilia Projects
1414
* Author URI: https://prpl.fyi/about
1515
* License: GPL-3.0+

readme.txt

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ Tags: planning, maintenance, writing, blogging
44
Requires at least: 6.3
55
Tested up to: 6.8
66
Requires PHP: 7.4
7-
Stable tag: 1.6.0
7+
Stable tag: 1.6.1
88
License: GPL3+
99
License URI: https://www.gnu.org/licenses/gpl-3.0.en.html
1010

@@ -110,6 +110,12 @@ https://youtu.be/e1bmxZYyXFY
110110

111111
== Changelog ==
112112

113+
= 1.6.1 =
114+
115+
Bugs we fixed:
116+
117+
* "Collect your point" not closing the "Test if your website can send emails correctly" popover.
118+
113119
= 1.6.0 =
114120

115121
Enhancements:

views/js-templates/suggested-task.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ <h3 style="width: 100%;">
2929
<# if ( data.post.content.rendered !== '' ) { #>
3030
<prpl-tooltip>
3131
<slot name="open-icon">
32-
<button type="button" class="prpl-suggested-task-button" data-task-id="{{ data.post.meta.prpl_task_id }}" data-task-title="{{ data.post.title.rendered }}" data-action="info" data-target="info" title="{{ data.l10n.info }}" onclick="prplSuggestedTask.runButtonAction( this );">
32+
<button type="button" class="prpl-suggested-task-button" data-task-id="{{ data.post.meta.prpl_task_id }}" data-task-title="{{ data.post.title.rendered }}" data-action="info" data-target="info" title="{{ data.l10n.info }}">
3333
<img src="{{ data.assets.infoIcon }}" alt="{{ data.l10n.info }}" class="icon">
3434
<span class="screen-reader-text">{{ data.l10n.info }}</span>
3535
</button>
@@ -53,7 +53,7 @@ <h3 style="width: 100%;">
5353
<# if ( data.post.meta.prpl_snoozable ) { #>
5454
<prpl-tooltip class="prpl-suggested-task-snooze">
5555
<slot name="open-icon">
56-
<button type="button" class="prpl-suggested-task-button" data-task-id="{{ data.post.meta.prpl_task_id }}" data-task-title="{{ data.post.title.rendered }}" data-action="snooze" data-target="snooze" title="{{ data.l10n.snooze }}" onclick="prplSuggestedTask.runButtonAction( this );">
56+
<button type="button" class="prpl-suggested-task-button" data-task-id="{{ data.post.meta.prpl_task_id }}" data-task-title="{{ data.post.title.rendered }}" data-action="snooze" data-target="snooze" title="{{ data.l10n.snooze }}">
5757
<img src="{{ data.assets.snoozeIcon }}" alt="{{ data.l10n.snooze }}" class="icon">
5858
<span class="screen-reader-text">{{ data.l10n.snooze }}</span>
5959
</button>

0 commit comments

Comments
 (0)