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

Portals-list fix #1169

Closed
wants to merge 3 commits into from
Closed

Portals-list fix #1169

wants to merge 3 commits into from

Conversation

McBen
Copy link
Contributor

@McBen McBen commented Jan 1, 2017

Portals-list fix for #1161

but 'cause of "Always leave the campground cleaner than you found it." I had to apply some more refactoring.

McBen added 3 commits January 1, 2017 11:40
- splitted the main function 'portalTable' into smaller parts.
- removed comments (old, obsolete, unnecessary)
@hayeswise
Copy link

hayeswise commented Jan 9, 2017

Before committing these changes, would you mind adding an id to the toolbox control? This will allow other plugins to create logical toolbox groupings of controls of other plugins.

An id example:

    $('#toolbox').append('<a id="portalslist-portalList" onclick="window.plugin.portalslist.displayPL()" title="Display a list of portals in the current view [t]" accesskey="t">Portals list</a>');

For the id, I used the pattern <plugin-object-name>-<control-name>. The plugin-object-name facilitates (but does not enforce) achieving unique ids across the plugins.

Thanks!

@pleasantone
Copy link
Contributor

Looks like we both had similar ideas. Please see #1190

@McBen
Copy link
Contributor Author

McBen commented Oct 19, 2022

closed because stale project

@McBen McBen closed this Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants