Differential D16683

[explorer] Remove extra spaces around 's'
ClosedPublic
Actions

Authored by johnkuney on Aug 27 2024, 03:48.

Tags

None

Subscribers

Details

Reviewers

PiRK
tobias_ruck

Group Reviewers

Restricted Project

Commits

rABCa3125926c643: [explorer] Remove extra spaces around 's'

Summary

on a transaciton page, in the block row under general details, the 's' in confirmations has a space either side
This came from the line break from the new formatting

Test Plan

preview the site and check the space is no longer there

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

johnkuney created this revision.Aug 27 2024, 03:48

Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 27 2024, 03:48

johnkuney requested review of this revision.Aug 27 2024, 03:48

Harbormaster completed remote builds in B30091: Diff 49361.Aug 27 2024, 03:48

johnkuney planned changes to this revision.Aug 27 2024, 03:49

fix spacing

Harbormaster completed remote builds in B30092: Diff 49362.Aug 27 2024, 03:57

PiRK added a subscriber: PiRK.Aug 27 2024, 13:12

PiRK added inline comments.

web/explorer/explorer-server/templates/pages/transaction.html
134 ↗	(On Diff #49362)	What does the dash do? Is there a link to some doc about this somewhere? It seems like it is freaking out the new linter (see extra indentation below)

adjust -

Harbormaster completed remote builds in B30102: Diff 49372.Aug 27 2024, 15:36

formatting

Harbormaster completed remote builds in B30103: Diff 49373.Aug 27 2024, 15:39

PiRK accepted this revision.Aug 27 2024, 15:47

This revision is now accepted and ready to land.Aug 27 2024, 15:47

actually this is still causing the space after the 's' looks like

Here is some not too helpful doc....https://djc.github.io/askama/configuration.html#whitespace-control

Im having trouble getting it to work and not add a bunch of new formatting to the rest of the page...

fix extra space

Harbormaster completed remote builds in B30104: Diff 49374.Aug 27 2024, 16:02

yeah this is the only way I can get it to work, but it tabs everything below it over

johnkuney requested review of this revision.Aug 27 2024, 16:15

tobias_ruck requested changes to this revision.Aug 27 2024, 21:21

tobias_ruck added a subscriber: tobias_ruck.

tobias_ruck added inline comments.

web/explorer/explorer-server/templates/pages/transaction.html
134 ↗	(On Diff #49374)	you can just repeat "confirmation" and not use {%-, which seems to confuse djlint

This revision now requires changes to proceed.Aug 27 2024, 21:21

fix spacing issue and avoid extra formatting

Harbormaster completed remote builds in B30118: Diff 49395.Aug 28 2024, 18:15

good call thanks, had to add the closing ) in each one as well

Works for me 👍🏻

Maybe in the future we can fix the formatter and simplify this again

This revision is now accepted and ready to land.Aug 28 2024, 19:24

Closed by commit rABCa3125926c643: [explorer] Remove extra spaces around 's' (authored by johnkuney). · Explain WhyAug 28 2024, 19:55

This revision was automatically updated to reflect the committed changes.

johnkuney added a commit: rABCa3125926c643: [explorer] Remove extra spaces around 's'.

Revision Contents
Changeset List

Path

Size

web/

explorer/

explorer-server/

templates/

pages/

transaction.html

8 lines

Diff 49397

web/explorer/explorer-server/templates/pages/transaction.html

Loading...