Skip to content
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

Portal History Ornaments #445

Open
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

MysticJay
Copy link
Contributor

@MysticJay MysticJay commented Feb 18, 2021

Ornaments for portal history
allows to reverse the ornament with a toggle button.
adopts color scheme and logic of "highlight portals by histroy" plugin.
SVG-Implementation and dialog by @EisFrei

plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
@johnd0e
Copy link
Contributor

johnd0e commented Mar 16, 2021

In general: I'm not sure about ornaments.
Also alternative solutions should be considered (https://github.com/EisFrei/IngressPortalHistoryFlags/)

@johnd0e johnd0e marked this pull request as draft March 17, 2021 08:59
@MysticJay MysticJay changed the title Portal History Highlighter & Ornaments Portal History Ornaments Mar 17, 2021
@MysticJay MysticJay added enhancement New feature or request plugin WIP Work in progress || Proof of the concept labels Mar 17, 2021
@MysticJay MysticJay marked this pull request as ready for review March 26, 2021 13:00
@MysticJay MysticJay requested a review from johnd0e March 26, 2021 13:30
@MysticJay MysticJay added Ready to merge and removed WIP Work in progress || Proof of the concept labels Mar 26, 2021
@MysticJay MysticJay force-pushed the History-Ornaments branch 2 times, most recently from 00811ca to 3c8edbc Compare March 26, 2021 19:15
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
plugins/portal-history-ornaments.js Outdated Show resolved Hide resolved
MysticJay and others added 9 commits July 13, 2021 20:15
Highlighter & Ornaments
for portal history
removed code for the highlighter
using own namespace
removing highlighter remnants
comment layer based code.
Exposed Functions
var vs let,const
unused code removed
Removing unused exposures
Removing unused functions & hooks

Co-Authored-By: Martin <[email protected]>
MysticJay and others added 21 commits July 13, 2021 20:16
style for Button not working properly.

Co-Authored-By: Martin <[email protected]>
hide toggle button when layer is inactive

Co-Authored-By: Martin <[email protected]>
Co-authored-by: johnd0e <[email protected]>
Various fixes to address reviews.
fixed toggleDisplayMode
added dblclick to button
function getSVGString

Co-Authored-By: LeJeu <[email protected]>
adding symbols to body
removed need for CSS
SVG-icond not reusable SVG symbols based on FontAwsome
Button and dialog now communicate
turning of the layer removes the button as well,

Co-Authored-By: McBen <[email protected]>
code corrections based on eslint

Co-Authored-By: McBen <[email protected]>
hide button on startup.

Co-Authored-By: McBen <[email protected]>
based on eslint
- changed names of expoed functions
- housekeeping
fixinf eslint complaints
layout and CSS changed.
Disabling this line. Seems not to work in IITCm
dialog() caused "undefined" error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants