search

Financial-Times / engineering-progression

Careers and progression for engineers in the CTO organisation.
https://engineering-progression.ft.com/
MIT License
116 stars 18 forks source link

Remove beta messaging from the site and docs #338

Closed rowanmanning closed 3 years ago

rowanmanning commented 3 years ago

We need to do this before releasing v1.0.0, which I'll do once this is merged. I decided to remove the beta banner rather than change it to announce the stable version - this turned out to be a pain to do (because o-message styling seems a bit broken) and I don't have the time to properly investigate now.

notlee commented 3 years ago

this turned out to be a pain to do (because o-message styling seems a bit broken)

@rowanmanning anything you'd like me to look into? if you don't mind sharing the message type/state you were aiming for

rowanmanning commented 3 years ago

Ha thanks Lee 😁 I think we may be using an older version of message, at some point I'll update everything but tbh I'm not sure if we need the message anyway.

If you're interested in having a look, it was the success modifier – the tick icon was overlaying the text.