[chore] Refactor HTML templates and CSS (#2480)

* [chore] Refactor HTML templates and CSS

* eslint

* ignore "Local"

* rss tests

* fiddle with OG just a tiny bit

* dick around with polls a bit more so SR stops saying "clickable"

* remove break

* oh lord

* don't lazy load avatar

* fix ogmeta tests

* clean up some cruft

* catch remaining calls to c.HTML

* fix error rendering + stack overflow in tag

* allow templating attributes

* fix indent

* set aria-hidden on status complementary content, since it's already present in the label anyway

* tidy up templating calls a little

* try to make styling a bit more consistent + readable

* fix up some remaining CSS issues

* fix up reports

1 files changed, 0 insertions, 180 deletions

diff --git a/internal/web/opengraph.go b/internal/web/opengraph.go
deleted file mode 100644
index 66b6c6eea..000000000
--- a/internal/web/opengraph.go
+++ /dev/null
@@ -1,180 +0,0 @@
-// GoToSocial
-// Copyright (C) GoToSocial Authors admin@gotosocial.org
-// SPDX-License-Identifier: AGPL-3.0-or-later
-//
-// This program is free software: you can redistribute it and/or modify
-// it under the terms of the GNU Affero General Public License as published by
-// the Free Software Foundation, either version 3 of the License, or
-// (at your option) any later version.
-//
-// This program is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-// GNU Affero General Public License for more details.
-//
-// You should have received a copy of the GNU Affero General Public License
-// along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-package web
-
-import (
-	"html"
-	"strconv"
-	"strings"
-
-	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"
-	"github.com/superseriousbusiness/gotosocial/internal/text"
-)
-
-const maxOGDescriptionLength = 300
-
-// ogMeta represents supported OpenGraph Meta tags
-//
-// see eg https://ogp.me/
-type ogMeta struct {
-	// vanilla og tags
-	Title       string // og:title
-	Type        string // og:type
-	Locale      string // og:locale
-	URL         string // og:url
-	SiteName    string // og:site_name
-	Description string // og:description
-
-	// image tags
-	Image       string // og:image
-	ImageWidth  string // og:image:width
-	ImageHeight string // og:image:height
-	ImageAlt    string // og:image:alt
-
-	// article tags
-	ArticlePublisher     string // article:publisher
-	ArticleAuthor        string // article:author
-	ArticleModifiedTime  string // article:modified_time
-	ArticlePublishedTime string // article:published_time
-
-	// profile tags
-	ProfileUsername string // profile:username
-}
-
-// ogBase returns an *ogMeta suitable for serving at
-// the base root of an instance. It also serves as a
-// foundation for building account / status ogMeta on
-// top of.
-func ogBase(instance *apimodel.InstanceV1) *ogMeta {
-	var locale string
-	if len(instance.Languages) > 0 {
-		locale = instance.Languages[0]
-	}
-
-	og := &ogMeta{
-		Title:       text.SanitizeToPlaintext(instance.Title) + " - GoToSocial",
-		Type:        "website",
-		Locale:      locale,
-		URL:         instance.URI,
-		SiteName:    instance.AccountDomain,
-		Description: parseDescription(instance.ShortDescription),
-
-		Image:    instance.Thumbnail,
-		ImageAlt: instance.ThumbnailDescription,
-	}
-
-	return og
-}
-
-// withAccount uses the given account to build an ogMeta
-// struct specific to that account. It's suitable for serving
-// at account profile pages.
-func (og *ogMeta) withAccount(account *apimodel.Account) *ogMeta {
-	og.Title = parseTitle(account, og.SiteName)
-	og.Type = "profile"
-	og.URL = account.URL
-	if account.Note != "" {
-		og.Description = parseDescription(account.Note)
-	} else {
-		og.Description = `content="This GoToSocial user hasn't written a bio yet!"`
-	}
-
-	og.Image = account.Avatar
-	og.ImageAlt = "Avatar for " + account.Username
-
-	og.ProfileUsername = account.Username
-
-	return og
-}
-
-// withStatus uses the given status to build an ogMeta
-// struct specific to that status. It's suitable for serving
-// at status pages.
-func (og *ogMeta) withStatus(status *apimodel.Status) *ogMeta {
-	og.Title = "Post by " + parseTitle(status.Account, og.SiteName)
-	og.Type = "article"
-	if status.Language != nil {
-		og.Locale = *status.Language
-	}
-	og.URL = status.URL
-	switch {
-	case status.SpoilerText != "":
-		og.Description = parseDescription("CW: " + status.SpoilerText)
-	case status.Text != "":
-		og.Description = parseDescription(status.Text)
-	default:
-		og.Description = og.Title
-	}
-
-	if !status.Sensitive && len(status.MediaAttachments) > 0 {
-		a := status.MediaAttachments[0]
-
-		og.ImageWidth = strconv.Itoa(a.Meta.Small.Width)
-		og.ImageHeight = strconv.Itoa(a.Meta.Small.Height)
-
-		if a.PreviewURL != nil {
-			og.Image = *a.PreviewURL
-		}
-
-		if a.Description != nil {
-			og.ImageAlt = *a.Description
-		}
-	} else {
-		og.Image = status.Account.Avatar
-		og.ImageAlt = "Avatar for " + status.Account.Username
-	}
-
-	og.ArticlePublisher = status.Account.URL
-	og.ArticleAuthor = status.Account.URL
-	og.ArticlePublishedTime = status.CreatedAt
-	og.ArticleModifiedTime = status.CreatedAt
-
-	return og
-}
-
-// parseTitle parses a page title from account and accountDomain
-func parseTitle(account *apimodel.Account, accountDomain string) string {
-	user := "@" + account.Acct + "@" + accountDomain
-
-	if len(account.DisplayName) == 0 {
-		return user
-	}
-
-	return account.DisplayName + " (" + user + ")"
-}
-
-// parseDescription returns a string description which is
-// safe to use as a template.HTMLAttr inside templates.
-func parseDescription(in string) string {
-	i := text.SanitizeToPlaintext(in)
-	i = strings.ReplaceAll(i, "\n", " ")
-	i = strings.Join(strings.Fields(i), " ")
-	i = html.EscapeString(i)
-	i = strings.ReplaceAll(i, `\`, "&bsol;")
-	i = trim(i, maxOGDescriptionLength)
-	return `content="` + i + `"`
-}
-
-// trim strings trim s to specified length
-func trim(s string, length int) string {
-	if len(s) < length {
-		return s
-	}
-
-	return s[:length]
-}