Module:Feedback: Difference between revisions

Browse history interactively

← Older edit

@@ Line 122: / Line 122: @@
 local function renderAttachments(entry)
-    local wrapper = mw.html.create('div')
+	local wrapper =
-        :addClass('feedback-attachments')
+		mw.html.create('div')
+			:addClass('feedback-attachments')
+	local link = trim(entry.link)
+	if link ~= '' then
+		local linkWrapper =
+			wrapper:tag('span')
+				:addClass('feedback-attachment-link')
+		linkWrapper:tag('span')
+			:addClass('feedback-attachment-icon')
+		linkWrapper:wikitext(
+			'[' .. link .. ' ' .. mw.text.nowiki(link) .. ']'
+		)
+	end
-    if entry.link and trim(entry.link) ~= '' then
+	local image = trim(entry.image)
-        local link = mw.html.create('a')
-            :addClass('feedback-link')
-            :attr('href', trim(entry.link))
-            :attr('target', '_blank')
-            :attr('rel', 'noopener noreferrer')
-            :wikitext('View link')
-        wrapper:node(link)
+	if image ~= '' then
-    end
+		image = mw.ustring.gsub(
+			image,
+			'^File:',
+			''
+		)
-    if entry.image and trim(entry.image) ~= '' then
+		local title =
-        local image = mw.html.create('img')
+			mw.title.new('File:' .. image)
-            :addClass('feedback-image')
-            :attr('src', trim(entry.image))
-            :attr('alt', 'Feedback image')
-            :attr('loading', 'lazy')
-        wrapper:node(image)
+		if title then
-    end
+			local imageWrapper =
+				wrapper:tag('span')
+					:addClass('feedback-attachment-image')
+			imageWrapper:tag('span')
+				:addClass('feedback-attachment-icon')
+			imageWrapper:wikitext(
+				'[' ..
+				title:fullUrl() ..
+				' ' ..
+				mw.text.nowiki(image) ..
+				']'
+			)
+		end
+	end
-    return tostring(wrapper)
+	return wrapper
 end
@@ Line 166: / Line 191: @@
 			:addClass('feedback-resolution')
-	resolution:wikitext('↳ Resolved by ')
+	resolution:wikitext('Resolved by ')
 	if resolvedBy ~= '' then
@@ Line 340: / Line 365: @@
 	-- Attachments
 	--------------------------------------------------------
-	renderAttachments(
+	content:node(
-		content,
+	    renderAttachments(entry)
-		entry
 	)

ReisuDesign (talk | contribs)