MVP finish

[franzenzenhofer]

• [x] able to import rules

• [x] (ideally: choose a (single) folder (rules still enable / disable / view))

• [x] new: no pre-check of rules anymore

• [x] new import rules / custom rules add further down (de-emphasizing of this feature)

• [x] upload with same rule as existing one should update the old one

• [x] view button for custom rules

• [x] no longer able to edit rules in browser (no new rule button, no duplicate rules, no edit)

• [x] new default position: right top (and snapped if at top right position)

• [x] default rules that were deleted (after app update) should vanish

• [x] change name of the extension to "Obtrusive Live Test" (change name of repository to "f19n-obtrusive-live-test"

• [x] general bug fixes

[franzenzenhofer]

hi moritz, added one thing

the official name of this extension is "Obtrusive Live Test"

if the old name "Obtrusive Livetest Chrome Extension" shows up anywhere visible, please change.

[neuling]

Panel should now snap to the corners of the browser. It also snaps to the bottom of the browser. What do you think @franzenzenhofer? Does it feel right? Can be disabled easily, just wanted to show it to you.

[neuling]

Most of the new view-only rule concept is finished. Possibility to select/upload multiple rules. No pre-check per rule. Removed functionality to edit and duplicate rules. Removed the ability to add an empty rule. Removed some unused packages.

Todo:

• [x] Beautify Rule-View (Highlight code, display rule name and add close button)
• [x] Move 'Add rule' down

[franzenzenhofer]

ad Panel should now snap to the corners of the browser

works and snapping to top and bottom right side is cool

but: now there is a lot of stuttering (see attachment) when moving around, if (easily) possible see if there is away to reduce this stuttering.

[franzenzenhofer]

custom rules should also have the view button

[franzenzenhofer]

otherwise looks good

[franzenzenhofer]

ad janking, passive event listeners? https://blogs.windows.com/msedgedev/2017/03/08/scrolling-on-the-web/#ehgHAiKlb6YqOca2.97

[franzenzenhofer]

hmmm, adding rules with the same name should update the existing URL, not add another rule with the same name (this made sense when we had the editor in there, but with now just adding URLs it does not make sense any longer)

[neuling]

i did some debugging about the stuttering. the iframe inside the panel causes some performance issues. without the iframe it works perfectly. not sure how to fix this atm will think about it.

[neuling]

removed the ace editor from the view-rule view and added syntax highlighting.

[neuling]

add rule button is now underneath the "custom rule" section. what do you think @franzenzenhofer? for me, thats the most intuitive location for this button.

[franzenzenhofer]

ad janking: not MVP critical, and yeah iframes are always an issue with rendering

[neuling]

unsupported rules should now be removed on install and/or extension update. to make sure the sync works also in production, we should test it once the extension is available on the chrome store.