ui: show alert when user flag (bookmark) pressed

[TheSecurityDev]

Description:

This PR implements showing an alert when the user flags a position on the route with the bookmark icon.

Reason for this feature:

There's currently very little feedback that occurs when the button is pressed. This makes it difficult to be sure that the button was pressed successfully.

Screenshot:

[github-actions]

Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:

• Convert your PR to a draft unless it's ready to review
• Read the contributing docs
• Before marking as "ready for review", ensure:
  • the goal is clearly stated in the description
  • all the tests are passing
  • the change is something we merge
  • include a route or your device' dongle ID if relevant

[github-actions]

UI Preview

All Screenshots

[incognitojam]

I think we might not want to change the border color for this - that has to always show the openpilot engagement state.

[TheSecurityDev]

I think we might not want to change the border color for this - that has to always show the openpilot engagement state.

That's fair, it was just one of the suggestions along with the feature request, and also I thought it would make more sense to match the timeline color that shows in connect.

It would definitely make it less complex to not do that.

[BBBmau]

would it be fine to include the yellow border around the bookmark button itself to make it more obvious?

[TheSecurityDev]

would it be fine to include the yellow border around the bookmark button itself to make it more obvious?

You mean when the button is clicked or all the time? That's a good idea though.

[MarcoTheDingo]

I think we might not want to change the border color for this - that has to always show the openpilot engagement state.

I'd be okay with just the pop-up alert saying it's flagged, that stays for ~2 sec. I just suggested the yellow border to match connect but it's not needed.

[github-actions]

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

[TheSecurityDev]

Bump

[adeebshihadeh]

Otherwise LGTM

[github-actions]

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

[TheSecurityDev]

Bump

[github-actions]

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

[TheSecurityDev]

sorry I went ahead and removed it too haha