search

minetest / contentdb

A content database for Minetest mods, games, and more
https://content.minetest.net
GNU Affero General Public License v3.0
93 stars 45 forks source link

make featured status in client more obvious #500

Closed wsor4035 closed 2 months ago

wsor4035 commented 6 months ago

Problem

(reporting from matrix) featured status in the minetest client isnt clear enough

Solutions

add [featured] to package title (suggested by roller)

Alternatives

🤷

Additional context

line of code that currently does it: https://github.com/minetest/contentdb/blob/36c63b46577a2198a2bc389b846b57d810793fe4/app/blueprints/api/endpoints.py#L90

ancientmarinerdev commented 6 months ago

It is worth considering changing background colour also, to help differentiate at a visual glance.

rubenwardy commented 6 months ago

I originally put this in the title, but found that it clipped on larger font sizes / smaller displays. The description wraps at least

It is worth considering changing background colour also, to help differentiate at a visual glance.

We can only change the screenshot, description, and title from the server-side. Other changes would need to be done on the engine, at which point I'd rather wait for the redesign

rubenwardy commented 6 months ago

I'm wondering if I can include colorize codes in the API response, or if Minetest will strip that out. If it's respected then we could give the featured label a different color

rubenwardy commented 2 months ago

Solved by https://github.com/minetest/contentdb/commit/465370d3fc433b1ae050090ad3e85196b610d6fc and https://github.com/minetest/minetest/pull/14510