CleverRaven/Cataclysm-DDA

Overhead lights over floors don't look good when surrounding floor tiles are sprited with multitile support in tileset

Open

#46 028 ouverte le 13 déc. 2020

Voir sur GitHub
 (3 commentaires) (2 réactions) (0 assignés)C++ (3 991 forks)batch import
<Bug>Good First IssueSDL: Tiles / Sound

Métriques du dépôt

Stars
 (9 264 stars)
Métriques de merge PR
 (Merge moyen 6j 5h) (292 PRs mergées en 30 j)

Description

Is your feature request related to a problem? Please describe.

When floors are drawn as miltitile sprites with borders (latest Ultica) these borders create visible squares around overhead lights with the same floor in description and thus in background.

screenshot

Describe the solution you'd like

Not sure if it's worth it, but adding a new https://github.com/CleverRaven/Cataclysm-DDA/blob/master/doc/JSON_INFO.md#connects_to might work.

Describe alternatives you've considered

Removing multitile support from floors in tileset sprites.

Additional context

https://github.com/I-am-Erk/CDDA-Tilesets/pull/320

Guide contributeur

Direction de recherche
Examinez le mécanisme connects to dans les définitions de tuiles et comment les lumières de plafond interagissent avec les sprites de sol multi tuiles. Envisagez de modifier le JSON de la tuile pour introduire un nouveau type de connexion qui empêche le rendu de la superposition lorsque des bordures multi tuiles sont présentes.
Stack technique
cpp
Domaine
game dev
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++JSON
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60