tangled.org / core
Monorepo for Tangled tangled.org
fork atom

appview/pages: remove @ from AtLink extension #909

merged opened by oppi.li targeting master from op/vtyvsnwqkrwr

we now prefer links with @. this change also moves the styling into the mention class entirely.

Signed-off-by: oppiliappan me@oppi.li

STACK 2
#910 appview/pages: use robust handle regex from indigo/syntax
3/3
0
#1
#909 appview/pages: remove @ from AtLink extension
1/3
0
#1
Labels

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:qfpnj4og54vl56wngdriaxug/sh.tangled.repo.pull/3maihgf2bpe22
+3 -3
Diff #1
unified split
+2 -2
appview/pages/markup/extension/atlink.go
··· 101 101 102 102 func (r *atHtmlRenderer) renderAt(w util.BufWriter, source []byte, n ast.Node, entering bool) (ast.WalkStatus, error) { 103 103 if entering { 104 - w.WriteString(`<a href="/@`) 104 + w.WriteString(`<a href="/`) 105 105 w.WriteString(n.(*AtNode).Handle) 106 - w.WriteString(`" class="mention font-bold">`) 106 + w.WriteString(`" class="mention">`) 107 107 } else { 108 108 w.WriteString("</a>") 109 109 }
+1 -1
input.css
··· 162 162 } 163 163 164 164 .prose a.mention { 165 - @apply no-underline hover:underline; 165 + @apply no-underline hover:underline font-bold; 166 166 } 167 167 168 168 .prose li {

History

2 rounds 0 comments
sign up or login to add to the discussion
oppi.li submitted #1
interdiff
1 commit
expand
026c207f
appview/pages: remove @ from AtLink extension
1/3 failed, 1/3 timeout, 1/3 success
expand
build.yml
fmt.yml
test.yml
expand 0 comments
pull request successfully merged
oppi.li submitted #0
diff
1 commit
expand
a20d1dbe
appview/pages: remove @ from AtLink extension
3/3 success
expand
build.yml
fmt.yml
test.yml
expand 0 comments