refactor(tpl): simplify sub item class name

marjune163 · marjune163 · commit dae32eb0a660 · 2020-01-23T16:22:51.000+08:00
BREAKING CHANGE

Sub items' class names of a directory are changed. This may break
external customized templates.

Before:

Class names are on &lt;a&gt; elements, which are children of &lt;li&gt; elements.
directory class names: "item", "item-dir"
file class names: "item", "item-file"

After:

Class names are on &lt;li&gt; elements.
directory class name: "dir"
file class name: "file"
diff --git a/src/tpl/page.html b/src/tpl/page.html
@@ -12,7 +12,7 @@
 
 	<link rel="stylesheet" type="text/css" href="{{.RootRelPath}}/../assert/main.css"/>
 </head>
-<body>
+<body class="{{if .IsRoot}}root-dir{{else}}sub-dir{{end}}">
 
 <ol class="path-list">
 	{{range .Paths}}
@@ -38,18 +38,18 @@
 {{end}}
 
 <ul class="item-list">
-	<li>
+	<li class="dir parent">
 		<a href="{{if .IsRoot}}./{{else}}../{{end}}">
 			<span class="name">../</span>
 			<span class="size"></span>
 			<span class="time"></span>
 		</a>
 	</li>
 	{{range .SubItems}}
-		{{$isDir := .IsDir}}
-		<li>
-			<a href="{{$subItemPrefix}}{{.Name}}{{if $isDir}}/{{end}}" class="item {{if $isDir}}item-dir{{else}}item-file{{end}}">
-				<span class="name">{{fmtFilename .Name}}{{if $isDir}}/{{end}}</span>
+		{{$isDir := .IsDir}}{{$subItemName := .Name}}
+		<li class="{{if $isDir}}dir{{else}}file{{end}}">
+			<a href="{{$subItemPrefix}}{{$subItemName}}{{if $isDir}}/{{end}}">
+				<span class="name">{{fmtFilename $subItemName}}{{if $isDir}}/{{end}}</span>
 				<span class="size">{{if not $isDir}}{{fmtSize .Size}}{{end}}</span>
 				<span class="time">{{fmtTime .ModTime}}</span>
 			</a>
diff --git a/src/tpl/page.html.go b/src/tpl/page.html.go
@@ -21,7 +21,7 @@ const pageTplStr = `
 <title>{{.Path}}</title>
 <link rel="stylesheet" type="text/css" href="{{.RootRelPath}}?assert=main.css"/>
 </head>
-<body>
+<body class="{{if .IsRoot}}root-dir{{else}}sub-dir{{end}}">
 <ol class="path-list">
 {{range .Paths}}
 <li><a href="{{.Path}}">{{.Name}}</a></li>
@@ -43,18 +43,18 @@ const pageTplStr = `
 </div>
 {{end}}
 <ul class="item-list">
-<li>
+<li class="dir parent">
 <a href="{{if .IsRoot}}./{{else}}../{{end}}">
 <span class="name">../</span>
 <span class="size"></span>
 <span class="time"></span>
 </a>
 </li>
 {{range .SubItems}}
-{{$isDir := .IsDir}}
-<li>
-<a href="{{$subItemPrefix}}{{.Name}}{{if $isDir}}/{{end}}" class="item {{if $isDir}}item-dir{{else}}item-file{{end}}">
-<span class="name">{{fmtFilename .Name}}{{if $isDir}}/{{end}}</span>
+{{$isDir := .IsDir}}{{$subItemName := .Name}}
+<li class="{{if $isDir}}dir{{else}}file{{end}}">
+<a href="{{$subItemPrefix}}{{$subItemName}}{{if $isDir}}/{{end}}">
+<span class="name">{{fmtFilename $subItemName}}{{if $isDir}}/{{end}}</span>
 <span class="size">{{if not $isDir}}{{fmtSize .Size}}{{end}}</span>
 <span class="time">{{fmtTime .ModTime}}</span>
 </a>
