maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Multiple Reactions: opens drawer on single tap - mobile device (#19155)

Browse source 
* Opens tap on single tap too

* Multiple Reactions single tap issue fixes

* Updated drawer heart+ icon behaviour
This commit is contained in:
Rajat Talesra 2023-02-22 16:15:17 +05:30 • committed by GitHub
parent 561ed1858f
commit 7eae421a2a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
app/assets/javascripts/initializers/initializeArticleReactions.js
View file

@ -215,15 +215,13 @@ function openDrawerOnHover() {
}
drawerTrigger.addEventListener('click', function (event) {
event.preventDefault();
var articleId = document.getElementById('article-body').dataset.articleId;
reactToArticle(articleId, 'like');
drawerTrigger.parentElement.classList.add('open');
});
if (isTouchDevice()) {
drawerTrigger.addEventListener('click', function () {
var articleId = document.getElementById('article-body').dataset.articleId;
reactToArticle(articleId, 'like');
});
watchForLongTouch(drawerTrigger);
drawerTrigger.addEventListener('longTouch', function () {
drawerTrigger.parentElement.classList.add('open');
@ -250,9 +248,10 @@ function openDrawerOnHover() {
function closeDrawerOnOutsideClick() {
document.addEventListener('click', function (event) {
const reactionElement = document.querySelector('.reaction-drawer');
if (reactionElement) {
const isClickInside = reactionElement.contains(event.target);
const reactionDrawerElement = document.querySelector('.reaction-drawer');
const reactionDrawerTriggerElement = document.querySelector('#reaction-drawer-trigger');
if (reactionDrawerElement) {
const isClickInside = reactionDrawerElement.contains(event.target) || reactionDrawerTriggerElement.contains(event.target);
if (!isClickInside) {
document.querySelector('.hoverdown.open').classList.remove('open');