abrignoni / abrignoni/iLEAPP

bug: html formats cancelled out with media column

未关闭
#1,202 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
bug
主要语言
Python
星标
1.2k
派生
303
平均合并
4 小时 32 分钟
30 天内合并 PR
216

描述

https://github.com/abrignoni/iLEAPP/blob/92e095f60eec073203f16b1586ddf5825a59b0d1/scripts/ilapfuncs.py#L339-L354C49

theres a bug where if you include a media column as part of a data set, it will wipe out any html injected formatting.

line 341 splits the data set tuples into plain and html. then we check for media types on line 352. if so, then line 354 splits a new data set pair but it uses the `data_list` object from a few lines up that has been replaced with plain text during the earlier split.

my suggestion is we do this media detection inside the html output if block. the html output has some limitations and restrictions and needs some of this formatting and prep work to display things well. then inside the html output if block, we can use just the html data set since we fill that value above regardless of injected html tags or not.

lava does not need the style information so we can ignore that from the media column header. lava has its own style rules to apply.

see PR https://github.com/abrignoni/iLEAPP/pull/1139 for references

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。