Ps/frontend fixes #177
Ps/frontend fixes #177
Conversation
|
This pull request introduces 1 alert and fixes 3 when merging 59540fb into 5f56318 - view on LGTM.com new alerts:
fixed alerts:
|
asfaltboy
force-pushed
the
ps/119/fix-null-field
branch
from
65a91f8
to
f1fe314
Compare
|
This pull request introduces 1 alert and fixes 3 when merging 59540fb into 65a91f8 - view on LGTM.com new alerts:
fixed alerts:
|
|
This pull request introduces 1 alert and fixes 3 when merging 59540fb into 43f3503 - view on LGTM.com new alerts:
fixed alerts:
|
+ update code to use v4
+ simplify inline tabular classes
add an empty form, only if not submitting and extras required
asfaltboy
force-pushed
the
ps/frontend-fixes
branch
from
59540fb
to
59544cd
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| // options added from the backend need to be removed | ||
| elm.find("option").remove(); | ||
| } | ||
| } |
Check notice
Code scanning / CodeQL
Semicolon insertion Note
| }, | ||
| _getCloseBtn = function(type) { | ||
| if(type !== _currPopupType || !mfp.currTemplate.closeBtn) { | ||
| mfp.currTemplate.closeBtn = $( mfp.st.closeMarkup.replace('%title%', mfp.st.tClose ) ); |
Check warning
Code scanning / CodeQL
Unsafe jQuery plugin Medium
| $('html').css(windowStyles); | ||
|
|
||
| // add everything to DOM | ||
| mfp.bgOverlay.add(mfp.wrap).prependTo( mfp.st.prependTo || $(document.body) ); |
Check warning
Code scanning / CodeQL
Unsafe jQuery plugin Medium
| _mfpTrigger('FirstMarkupParse', markup); | ||
|
|
||
| if(markup) { | ||
| mfp.currTemplate[type] = $(markup); |
Check warning
Code scanning / CodeQL
Unsafe jQuery plugin Medium
fix #169
fix #38