[CmdPal] Removing App tag

[niels9001]

The "App" tag is pretty redundant and is adding a lot of visual noise. Closes #38968

New:

[moooyo]

Agree.

Another thing is, in top level page, only Apps result have app tags.

I think we also need to add tag for fallback item to distinguish their source?

[zadjii-msft]

Wait doesn't this also get rid of the top-level tag too? I thought we wanted that to make sure that we could clearly differentiate the app results from the command ones

[niels9001]

@moooyo @zadjii-msft Meh.. that's a good point. Is there a way to differentiate between the two?

Still, even on a toplevel, the default tag UX feels 'heavy' to just point something out as a tag.. Not for this PR, but maybe something to figure out longer term how make that more subtle

[zadjii-msft]

gah this is annoying. There's not an easy way to differentiate them - least not currently. The main page needs ListItems with tags. The Apps page doesn't. That basically necessitates having two different lists of ListItems, one with tags and one without.

seems like it'd be silly for us to be constructing new ListItem's as we're filtering the top-level. That should be the hottest, cleanest path

I suppose the cache could have them all be created with the tags, and then have AllAppsPage.GetItems iterate over the cache, and build new ListItems specific to that page, without the tags.