Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

Added shortcut buttons to "Last seen" members filter datepicker #2291

Merged

Conversation

kevinansfield
Copy link
Collaborator

refs https://github.com/TryGhost/Team/issues/1410

  • added block invocation support to <GhDatePicker> that exposes the calendar components used inside the datepicker dropdown allowing for customisation of the dropdown content
  • updated "last seen" datepicker to use the block invocation support to add buttons that select typical "time ago" dates

refs TryGhost/Product#1410

- added block invocation support to `<GhDatePicker>` that exposes the calendar components used inside the datepicker dropdown allowing for customisation of the dropdown content
- updated "last seen" datepicker to use the block invocation support to add buttons that select typical "time ago" dates
@codecov-commenter
Copy link

codecov-commenter commented Mar 4, 2022

Codecov Report

Merging #2291 (f501c48) into main (dcef9ea) will increase coverage by 0.00%.
The diff coverage is 80.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2291   +/-   ##
=======================================
  Coverage   41.18%   41.19%           
=======================================
  Files         579      579           
  Lines       22779    22782    +3     
  Branches     4028     4030    +2     
=======================================
+ Hits         9382     9384    +2     
- Misses      13397    13398    +1     
Impacted Files Coverage Δ
app/components/gh-date-picker.js 94.93% <80.00%> (-1.12%) ⬇️
app/components/gh-image-uploader.js 94.66% <0.00%> (-0.67%) ⬇️
app/components/gh-notification.js 100.00% <0.00%> (+4.54%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dcef9ea...f501c48. Read the comment docs.

@kevinansfield kevinansfield marked this pull request as ready for review March 4, 2022 17:08
@kevinansfield kevinansfield merged commit b948054 into TryGhost:main Mar 4, 2022
@kevinansfield kevinansfield deleted the members-last-seen-shortcuts branch March 4, 2022 17:09
kevinansfield added a commit to kevinansfield/Ghost-Admin that referenced this pull request Mar 8, 2022
…cuts

closes TBD

- drops the block usage of `<GhDatePicker>` for the "Last seen" input that added shortcut buttons
  - not a full revert of TryGhost#2291 as allowing block usage of the component is still useful for other component customisation
- removes CSS related to the calendar-with-shortcut buttons as we don't have any other use-cases for it at the moment
kevinansfield added a commit to kevinansfield/Ghost-Admin that referenced this pull request Mar 8, 2022
…cuts

refs TryGhost/Product#1418

- drops the block usage of `<GhDatePicker>` for the "Last seen" input that added shortcut buttons
  - not a full revert of TryGhost#2291 as allowing block usage of the component is still useful for other component customisation
- removes CSS related to the calendar-with-shortcut buttons as we don't have any other use-cases for it at the moment
kevinansfield added a commit that referenced this pull request Mar 8, 2022
…cuts (#2296)

refs TryGhost/Product#1418

- drops the block usage of `<GhDatePicker>` for the "Last seen" input that added shortcut buttons
  - not a full revert of #2291 as allowing block usage of the component is still useful for other component customisation
- removes CSS related to the calendar-with-shortcut buttons as we don't have any other use-cases for it at the moment
tigefa4u pushed a commit to tigefa4u/Ghost that referenced this pull request Aug 3, 2022
…cuts (TryGhost#2296)

refs TryGhost/Product#1418

- drops the block usage of `<GhDatePicker>` for the "Last seen" input that added shortcut buttons
  - not a full revert of TryGhost/Admin#2291 as allowing block usage of the component is still useful for other component customisation
- removes CSS related to the calendar-with-shortcut buttons as we don't have any other use-cases for it at the moment
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants