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

Sticky Incidents #64

Open
jbrooksuk opened this issue Oct 3, 2024 · 4 comments
Open

Sticky Incidents #64

jbrooksuk opened this issue Oct 3, 2024 · 4 comments
Labels
Milestone

Comments

@jbrooksuk
Copy link
Member

Incidents can be marked as “Sticky” which I think actually makes more sense to just be labelled as “Pin to top”. Logic needs implementing for this.

  1. Update language string from "Sticky" to "Pin to Top".
  2. Any "Sticky" incidents should be at the top of the timeline.
@jbrooksuk jbrooksuk added this to the v3.0 milestone Oct 3, 2024
@mechelon
Copy link
Contributor

mechelon commented Oct 4, 2024

Should the pinned incidents still show up in the regular list on their respective date?

@jbrooksuk
Copy link
Member Author

That's a good question! I think yes, but with some indicator that they're pinned too.

Though, that might be weird if you have only one incident and it's pinned 🤔

@mechelon
Copy link
Contributor

mechelon commented Oct 7, 2024

I can imagine both ways 🤔 And yes, a pin indicator makes sense.
Or we leave it to the user and add, in case pinning is enabled, another toggle for "Also show in timeline", which is enabled per default?

@jbrooksuk
Copy link
Member Author

@mechelon yeah, that's a good suggestion!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants