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

Add plugin support and fix image size #162

Merged
merged 4 commits into from
Nov 22, 2023

Conversation

byrnHDF
Copy link
Collaborator

@byrnHDF byrnHDF commented Sep 27, 2023

add user option entry for plugin path
add properties to save plugin path
appends path, but needs to be changed to replacing the path so every open table doesn't increase the number of path variables.
Address #117

@byrnHDF byrnHDF added Type - Improvement Improvements that don't add a new feature or functionality Component - Build Build system improvement Component - Object Library Improvements to the object library Component - HDFView Improvements to the visual interface layer Priority - 0. Blocker ⛔ This MUST be merged for the release to happen labels Sep 27, 2023
@byrnHDF byrnHDF requested a review from derobins September 27, 2023 20:37
@byrnHDF byrnHDF self-assigned this Sep 27, 2023
@byrnHDF
Copy link
Collaborator Author

byrnHDF commented Sep 27, 2023

The UserOption dialog for plugin path, should probably be converted to a list of the paths and allow the user to edit the list?


pluginDir = ViewProperties.getPluginDir();
if (pluginDir == null)
pluginDir = rootDir;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be rootDir + "/plugin" to match the default in ViewProperties.java?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably, it should just be a default if none are present, situation.

currentDirButton.setLayoutData(new GridData(SWT.FILL, SWT.FILL, false, false));
currentDirButton.addSelectionListener(new SelectionAdapter() {
@Override
public void widgetSelected(SelectionEvent e) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably just let users select the plugin dir rather than a base directory. I think that will be confusing to a lot of people

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have been thinking more about this, since this proof of implementation. It needs to be a list of paths and editable. At which time I think I will just drop that extra "/plugin" and require the real value.

@byrnHDF byrnHDF merged commit a8e7851 into HDFGroup:master Nov 22, 2023
@byrnHDF byrnHDF deleted the master-add-plugins branch December 13, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - Build Build system improvement Component - HDFView Improvements to the visual interface layer Component - Object Library Improvements to the object library Priority - 0. Blocker ⛔ This MUST be merged for the release to happen Type - Improvement Improvements that don't add a new feature or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants