Allow to disable particular items in ObjectPopup

[denis-bunakalya]

Contribution checklist

Brief description

Checklist

• - UI test added to added/changed functionality?
• - Screenshot is added to PR if applicable ?
• - Does the code work? Check whether function and logic are correct.
• - Does Changelog.md is updated with changes?
• - Does the translations are up to date?
• - Does it well tested?
• - Tested for Chrome.
• - Tested for Safari.
• - Go through the changed code looking for typos, TODOs, commented LOCs, debugging pieces of code, etc.
• - Rebase your branch onto master and upstream branch
• - Is there any redundant or duplicate code?
• - Are required links are linked to PR?
• - Does new code is well documented ?

Related issues

A list of closed updated issues

[haiodo]

@denis-bunakalya I do not like this idea at all.

[denis-bunakalya]

@denis-bunakalya I do not like this idea at all.

Any reason? We need it to show all the selected items, but disable editing for some of them. If we would show only editable items, user sometimes won't see other previously selected items, which are not editable for him

[haiodo]

So you need to disallow deselection of some of items?
In this case you need name it differently, since it is confusing.

And potentially it should be shown differently in control.

[denis-bunakalya]

So you need to disallow deselection of some of items? In this case you need name it differently, since it is confusing.

And potentially it should be shown differently in control.

Done

[haiodo]

Suggested change

	export let forbiddenDeselectItemIds: Set<Ref<Doc>> = new Set()
	export let disallowDeselect: Ref<Doc>[]| undefined

[denis-bunakalya]

Done