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

Replace isNew with generic, filterable context #278

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

Conversation

MatthiasKainer
Copy link

tl;dr

After this change, instead of having a fixed isNew, the technology radar would contain a filterable context field. This would be a breaking change

Rationale

For our usage of the technology radar, isNew doesn't provide that much value, as the new items we can get from the git history for the csv file. As we have a lot of blips, though, we needed a way of clustering and filtering them

Solution

By replacing isNew with context, we can use more values, or oeven keep the isNew functionality. I did decide on no longer supporting isNew, as I felt this would add complexity for downwards compatibility which wouldn't make a lot of sense in the long run. If you feel that compatibility is important let me know, and I'll add a commit that triggers the old functionality in case isNew is in the header, still.

The context values are read, and will create colored blips for up to 4 different contexts. Additionally, it will create a filter element in the header, which will highlight the respective blips.

See an example here:

Filterable technology radar

Benefits

  • Opens up new opportunities for displaying contextual information on the radar
  • Even for the then-replaced usecase of isNew, will allow filtering on the radar, thus improving usability

Trade-Offs

  • Breaking change
  • The additional menu item fills up space
  • As the color-palette is limited, same colors are used for different meanings
  • context is not really a good name

Super interested to hear your thoughts on this change

@devansh-sharma-tw
Copy link
Contributor

Hi @MatthiasKainer ,

Thanks for raising this PR!
Sorry for the delay in responding back. We are currently in discussion with the team regarding this PR and will update here shortly.

Thanks!

@marisahoenig
Copy link
Member

Hi @aksatpute, we've seen folks use multiple tabs for different categories to do something similar. However, this would display each category as a separate radar. At this time, we don't plan to add filtering functionality, as that is not currently a feature on the Technology Radar. If there are many people who would find this feature useful and a PR that's brought up to date with the new design, we may be able to prioritize it.

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.

3 participants