-
Notifications
You must be signed in to change notification settings - Fork 4
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
add(some change in main.css (ar)) #319
Conversation
הי אליסה, |
…rntley the animation has a bug on supported s)
@mayPeleg @AlisaZil
ההפרדה למשימות תסייע לנהל את הקוד, הפידבקים, ואת הפיתוח בצורה חלקה יותר, ולהבין את סטטוס ההתקדמות בצורה נגישה יותר |
opendPopup = true; | ||
}); | ||
}else{ | ||
arClick = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ההתייחסות פה היא רק לאייפון?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
הוא בודק אם זה אייפון
ואם לא הוא פשוט מעביר לעמוד בלי פונקציונליות
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
לעמוד ar בלי פונקציונליות נכון? לשנות את ה -arClick ל -false מבלבל קצת
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
אתקן בpr החדש מצאתי פונקציונליות יותר קלה שזה לבדוק אם ה chekbok בחור או לא
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
סוכם בדיון - זה יבוצע בPR נפרד
ar.html
Outdated
</svg> | ||
|
||
</div> | ||
<p id="url-copied">הקישור הועתק! פרסם את זה עם חברים שלך</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
מגניב
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
נראה שעשית עבודה טובה,
לא בדקתי אם זה עובד - סומך עליך שבדקת וראית שלא נשבר כלום.
אשמח אם תחזירי את package-lock למה שהיה קודם לפני שנמזג למאסטר
שימי לב לקונפליקטים |
opendPopup = true; | ||
}); | ||
}else{ | ||
arClick = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
סוכם בדיון - זה יבוצע בPR נפרד
בין הפיתוחים השונים שנמצאים פה בPR בוצע פתרון של issue #325 לעמוד של הAR. |
need to change old ar css to a new css