impress/app/views/items/_item_list_row.html.haml

- subtitle = nil unless defined? subtitle

%tr
	%td.thumbnail-cell
		= link_to item_thumbnail_for(item), item, target: "_blank",
		  tabindex: "-1"
	%td.name-cell
		= link_to item.name, item, target: "_blank"
		- if subtitle.present?
			.subtitle= subtitle
	%td.actions-cell= yield
Oops, fix recent bug in Item Getting Guide for most item types Oh oops, I added an optional `subtitle` argument for the item table list row template, but didn't realize that it would crash in cases when not provided! Now, we add an initializer to set it to `nil` if undefined. 2024-05-28 17:05:28 -07:00			`- subtitle = nil unless defined? subtitle`

Refactor Item Getting Guide table rows to use a shared template I'm gonna add more stuff like linking to the item page and stuff, and that's starting to get complex enough to not want to repeat myself! 2024-05-21 18:48:29 -07:00			`%tr`
Improve row focus and keyboard nav in Item Getting Guide Clearer focus states, row changes bg color on hover/focus to help you track where you are, remove the redundant image thumbnail link from the tab order (it's the same as the item name link!) 2024-05-22 14:57:28 -07:00			`%td.thumbnail-cell`
			`= link_to item_thumbnail_for(item), item, target: "_blank",`
			`tabindex: "-1"`
Add OWLS values to non-NC-Mall NC items in Item Getting Guide Note that there's a known performance issue here: we should try to fetch all the OWLS values at once, instead of doing them in sequence while rendering the page! 2024-05-27 15:50:41 -07:00			`%td.name-cell`
			`= link_to item.name, item, target: "_blank"`
			`- if subtitle.present?`
			`.subtitle= subtitle`
Refactor Item Getting Guide table rows to use a shared template I'm gonna add more stuff like linking to the item page and stuff, and that's starting to get complex enough to not want to repeat myself! 2024-05-21 18:48:29 -07:00			`%td.actions-cell= yield`