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

Update BYOR to support moved in/out blip styles #177

Closed
setchy opened this issue Apr 13, 2021 · 22 comments
Closed

Update BYOR to support moved in/out blip styles #177

setchy opened this issue Apr 13, 2021 · 22 comments

Comments

@setchy
Copy link
Contributor

setchy commented Apr 13, 2021

Long time follower of the bi-yearly ThoughtWorks Radar publications. Hugely insightful and a great asset to all.

We have our own internal radar at my organization. I'd love to adopt the updated presentation styling as displayed on https://www.thoughtworks.com/radar.

One of the main styling difference is the public radar blips use 3 statuses

  • new
  • moved in/out
  • no change

As opposed to the version in this project which is only 2 statuses

  • new or moved
  • no change
@setchy
Copy link
Contributor Author

setchy commented Oct 27, 2021

Re-bumping this issue to coincide with the public release earlier today of Thoughtworks Tech Radar volume 25 - thoughtworks.com/radar

I really like the updated style used for the quadrants, blip shapes and the moved in/out symbol, which on closer observation noticed that it subtly varies by quadrant to show the in or out movements. Neat!

@sidyatish - do you know if this would be possible? Would be happy to pair on a PR if you needed help

@elenigen
Copy link

@Vaishnavij Is there any intention at Thoughtworks to share this feature (moved in/out) to this public repository?

@Vaishnavij
Copy link
Contributor

Hello @elenigen ,

We are discussing this internally. We will let you know as soon as we arrive at a decision. Thank you so much for the feature request and your patience.

Thanks,
BYOR Team

@elenigen
Copy link

Thank you for taking the time to reply 😄 I usually don't tag people, but I just noticed it was open since April so I was hoping we could know if eventually this feature would be available or not, since I'm about to build a tech radar and I would really enjoy to use this feature.

@Vaishnavij
Copy link
Contributor

Hello @elenigen,

Thanks for waiting for us. We have good news!
We(Team + Stakeholders) accept this will be a great addition to BYOR tool . We have started working on identifying the changes that we need to do to the tool itself to accommodate this feature.

Will keep you posted with further updates as we get it.

Thanks,
Vaishnavi Jayakumar
BYOR Team

@setchy
Copy link
Contributor Author

setchy commented Nov 25, 2021

This is great news @Vaishnavij. Thank you TW BYOR Team

@elenigen
Copy link

Thanks a lot for taking the time to share this good news with us! Looking forward to adopt this new feature. 😃

@elenigen
Copy link

Is this feature will be my Christmas 🎁 🎅?

@setchy
Copy link
Contributor Author

setchy commented Feb 1, 2022

Hi @Vaishnavij and BYOR Team - would you happen to have any new updates re this enhancement?

@Vaishnavij
Copy link
Contributor

@setchy : We are working on bringing a new skin to BYOR (that is slightly different from Official Radar) with our design team and stakeholders. We will keep you posted as we start the development.

@elenigen
Copy link

elenigen commented May 3, 2022

Hi @Vaishnavij, just checking if there's some progress on this feature, thanks!

@Vaishnavij
Copy link
Contributor

Hello @elenigen and @setchy,

Hope you are doing well!
The plan currently is to pick up the development for the features in the upcoming weeks and release it by early September.
Thanks for your patience!

Thanks,
Vaishnavi Jayakumar
BYOR Team

@elenigen
Copy link

Thank you for the update and sharing the target with us! 😃

@elenigen
Copy link

elenigen commented Sep 6, 2022

@Vaishnavij We are already early September, any good news for us? 🙏

@devansh-sharma-tw
Copy link
Contributor

Hi @elenigen , we are working on updating the design to match the TW Radar design (and some other UX updates as well) and are in the middle of the development and testing. Unfortunately, we had some other tasks come up which the team had to prioritize over this, and so this is getting delayed.

@elenigen
Copy link

elenigen commented Sep 7, 2022

@devansh-sharma-tw Thank you for the update! :)

@Javabien
Copy link

Javabien commented Feb 7, 2023

Hi @devansh-sharma-tw would be possible to share an update on this, is it still on hold?

@setchy
Copy link
Contributor Author

setchy commented Feb 24, 2023

Looks like a new wave of UI changes have been merged to master which is very exciting 🎉

I've pulled the latest master and run it locally (npm run dev). The full radar view works, but the quadrant buttons or drilling into a specific quadrant does not render the chart. npm run dev:old-ui still functional.

@devansh-sharma-tw
Copy link
Contributor

Thanks for mentioning this @setchy !
@Javabien To confirm, yes, we are working on updating the UI currently. The plan is to roll out the new UI to radar.thoughtworks.com shortly, once all components/screens are updated with the new design. By that time, master branch would also have the new design code.

Thanks!

@setchy
Copy link
Contributor Author

setchy commented Apr 20, 2023

Great work Thoughtworks team // @devansh-sharma-tw 👏 - v1.0.0 looks really nice - can't wait to adopt it.

I'm going to suggest leaving this issue open as the main feature enhancement I was after is the moved in/out legend styling / logic.

Is this something that could be easily added into build-your-own-radar now that the major design overhaul is complete?

Perhaps it could be done in such a way that allows either based on the data provided (eg: two new optional column called hasMovedIn, hasMovedOut)

  • two category legend: new, existing
  • three category legend: new, moved in/out, no change

@setchy setchy changed the title Update BYOR to match style of public TW Radar Update BYOR to support moved in/out blip styles Apr 20, 2023
@setchy
Copy link
Contributor Author

setchy commented Apr 22, 2023

Raised #310 to contribute support for drawing moved in / out blips

@tardich
Copy link

tardich commented Sep 12, 2023

Hello. Any progress on this request?

@setchy setchy closed this as completed Jan 14, 2025
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

No branches or pull requests

6 participants