Skip to content

Commit

Permalink
Merge pull request #771 from Financial-Times/ELES-1161-include-highli…
Browse files Browse the repository at this point in the history
…ghts-share-option-is-not-checked-on-highlights-dashboard

ELES-1161 Preserve Include Highlights checkbox state through tabs
  • Loading branch information
ststdimov authored Apr 3, 2024
2 parents b1035b1 + b7799d0 commit d2a2627
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions components/x-gift-article/src/SharingOptionsToggler.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ export const SharingOptionsToggler = (props) => {
actions.showAdvancedSharingOptions()
} else {
actions.hideNonSubscriberSharingOptions(event)
actions.setIncludeHighlights(false)
actions.setIncludeHighlights(event.target.checked)
}
return
}
Expand Down Expand Up @@ -68,7 +68,7 @@ export const SharingOptionsToggler = (props) => {
<span className="o-forms-input__label">Non-subscriber</span>
</label>
)}
<label htmlFor="share-with-one-person-radio" className="no-margin">
<label htmlFor="share-with-subscribers-radio" className="no-margin">
<input
id="share-with-subscribers-radio"
name="share-option"
Expand Down

0 comments on commit d2a2627

Please sign in to comment.