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

Minor explorer tuning #430

Merged
merged 9 commits into from
Feb 8, 2021
Merged

Minor explorer tuning #430

merged 9 commits into from
Feb 8, 2021

Conversation

eputtone
Copy link
Member

@eputtone eputtone commented Feb 5, 2021

  1. Add hooks for customizing Explorer look through config.js (header logo, hide footer, change HTML title)
  2. New look for workflow instance properties (previous made your eyes bleed). Also changed the order a bit and removed redundant workflow instance id (it was shown after workflow definion name already)
  3. Layout of workflow instance properties and graph changed. Usually everything (properties, graph, tabs) fit "on the same line", but for large workflows (>15 states) rendering remains as it was (graph below workflow instance properties)
  4. Separate ugly link from workflow instance page removed and workflow type name converted to a link.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 80.941% when pulling 39e5615 on minor-explorer-tuning into 78a48a3 on master.

@coveralls
Copy link

coveralls commented Feb 5, 2021

Coverage Status

Coverage remained the same at 80.936% when pulling 55dd743 on minor-explorer-tuning into 78a48a3 on master.

@efonsell
Copy link
Member

efonsell commented Feb 8, 2021

Changelog?

@efonsell efonsell merged commit 3c5c587 into master Feb 8, 2021
@efonsell efonsell deleted the minor-explorer-tuning branch February 8, 2021 12:48
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.

None yet

3 participants