StardewValley
/
SMAPI
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Simplify visible section checking by abusing Vue behavior, since the proper way is being buggy.

This commit is contained in:
Khloe Leclair 2022-04-08 15:26:35 -04:00
parent 0beff189d1
commit 631d0375c3
2 changed files with 1 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/SMAPI.Web/Views/LogParser/Index.cshtml
View File

@ -404,7 +404,6 @@ else if (log?.IsValid == true)
v-bind:showScreenId="showScreenId"
v-bind:message="msg"
v-bind:highlight="shouldHighlight"
v-bind:sectionExpanded="msg.SectionName && visibleSections.includes(msg.SectionName)"
/>
</log-table>
}

13
src/SMAPI.Web/wwwroot/Content/js/log-parser.js
View File

@ -347,10 +347,6 @@ smapi.logParser = function (data, sectionUrl) {
type: Object,
required: true
},
sectionExpanded: {
type: Boolean,
required: false
},
highlight: {
type: Boolean,
required: false
@ -379,7 +375,7 @@ smapi.logParser = function (data, sectionUrl) {
const events = {};
let toggleMessage;
if (msg.IsStartOfSection) {
const visible = context.props.sectionExpanded;
const visible = msg.SectionName && window.app && app.sectionsAllow(msg.SectionName);
events.click = smapi.clickLogLine;
toggleMessage = visible ?
'This section is shown. Click here to hide it.' :
@ -477,13 +473,6 @@ smapi.logParser = function (data, sectionUrl) {
// Maybe not strictly necessary, but the Vue template is being
// weird about accessing data entries on the app rather than
// computed properties.
visibleSections: function () {
const ret = [];
for (const [k, v] of Object.entries(this.showSections))
if (v !== false)
ret.push(k);
return ret;
},
hideContentPacks: function () {
return !data.showContentPacks;
},