#1 - 2014-11-6 08:01
后圣域传说 (才能上的差距……还真是令人嫉妒)
http://bgm.tv/ep/432501#post_245436
在讨论串的多重回复下由于每次回复会向右缩进，从而导致显示图片大小一定的情况下越往后越会超出边框。
可以考虑在显示图片的时候适当缩放对齐右边框。
好吧其实就是我强迫症觉得不好看……
#2 - 2014-11-6 10:48
夜々無夢 (約束の言葉 大好きな花 ポッケの宝物～? ...)
其實還有這種情況：
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
#2-1 - 2014-11-6 11:21
后圣域传说
是的，救命！
#2-2 - 2014-11-6 15:13
夜々無夢
后圣域传说 说: 是的，救命！
再看怎麼裁短了？我再試試～
————————————————————————————————————————————————————————————————————————————————————
#2-3 - 2014-11-6 15:15
夜々無夢
夜々無夢 说: 再看怎麼裁短了？我再試試～
————————————————————————————————————————————————————————————————————————————————————
————————————————————————————！！！！！！！！！！！！！！！！！！！！！！！！！！！！♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞☞
#2-4 - 2018-3-8 00:14
后圣域传说
夜々無夢 说: ————————————————————————————！！！！！！！！！！！！！！！！！！！！！！！！！！！！♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂♂☞☞☞☞☞☞...
这个Bug修正啦！！！
#2-5 - 2018-3-8 00:39
谷米
后圣域传说 说: 这个Bug修正啦！！！
手机版未修正（
#2-6 - 2018-3-8 00:50
后圣域传说
咕咪 说: 手机版未修正（
我的理解是现在https://bgm.tv也能支持手机版了，所以https://bgm.tv/m就已经被放弃了。你直接访问https://bgm.tv就行了……
#2-7 - 2018-3-8 09:11
夜々無夢
后圣域传说 说: 这个Bug修正啦！！！
驚了
#3 - 2014-11-6 12:21
CMK (No Music, No Life)
动画评论里贴的 Endcard，大多都超边框…
#3-1 - 2018-3-8 00:14
后圣域传说
这个Bug修正啦！！！
#4 - 2018-3-8 00:13
后圣域传说 (才能上的差距……还真是令人嫉妒)
因为前段时间开放了组件功能，所以我就自己动手丰衣足食撸了个组件来解决讨论串回复超长单词和插入大图超出边框的问题（https://bgm.tv/dev/app/9）。
然而由于审核速度过于缓慢以及我孜孜不倦地催审（http://bgm.tv/group/topic/345157#post_1324350），最终Sai老板Review以后直接合并进主站修正了这两个Bug，好处显而易见，组件加载速度肯定没有CSS直接加载速度快，只是大家再也看不到我的这个组件了233……
时隔四年的一个修正😂
#4-1 - 2018-3-8 00:17
Rくん
Amazing！(bgm38)
因为占用面积特别大，而且会超出边框。我平时贴图就是规定像素数，做个成比例缩放。
#4-2 - 2018-3-8 00:24
后圣域传说
Rくん 说: Amazing！
因为占用面积特别大，而且会超出边框。我平时贴图就是规定像素数，做个成比例缩放。
赞美注意细节的贴图回复！
作为强迫症我看到图片超出边框就特别难受，但是不是所有人都有精力再对图片进行缩放，大部分人都是直接720P或者1080P的截图放上去必然就超了，而且很多动图再缩放一下就不好看了。本着严以律己，宽以待人的态度，我只能自己写个CSS来修正这个问题，终于开放了组件功能然后直接修正了我真是泪流满面……
#4-3 - 2018-3-8 00:25
Sai🖖
后圣域传说 说: 赞美注意细节的贴图回复！
作为强迫症我看到图片超出边框就特别难受，但是不是所有人都有精力再对图片进行缩放，大部分人都是直接720P或者1080P的截图放上去必然就超了，而且很多动图再缩放一下就不好看了...
可以做一个 hover 放大成原始尺寸的组件（
#4-4 - 2018-3-8 01:10
ℰ𝓊𝓅𝒽𝓎
Sai 说: 可以做一个 hover 放大成原始尺寸的组件（
后排支持
#4-5 - 2018-3-9 13:35
后圣域传说
Sai 说: 可以做一个 hover 放大成原始尺寸的组件（
既然Sai老板都这么说了，我就又撸了两行代码写了个组件……这次审核要等多久呢（
#4-6 - 2018-3-9 13:50
后圣域传说
后圣域传说 说: 既然Sai老板都这么说了，我就又撸了两行代码写了个组件……这次审核要等多久呢（
给跪了，这审核通过速度……真快啊……
#5 - 2018-3-11 19:32
chitanda＠Lv2 (Make Bangumi Great Again!)
话说现在这个hover显示原图尺寸的效果有点鬼畜了。。
图片较大的话本来一个浏览器界面内也放不下。还不如限制最大不超过当前屏幕并居中的显示效果更好点。。
#5-1 - 2018-3-11 21:29
Thunder
+1，目前这个版本还是不太实用
#5-2 - 2018-3-12 00:06
chitanda＠Lv2
小鸠世界第一可爱 说: +1，目前这个版本还是不太实用
主要这个hover出来的原图都不只是简单的"超出边框"的范畴了。。
#5-3 - 2018-3-12 00:15
Thunder
chitanda@Lv2 说: 主要这个hover出来的原图都不只是简单的"超出边框"的范畴了。。
冲出屏幕的震撼视觉体验(bgm38)
#5-4 - 2018-3-12 08:33
后圣域传说
非常感谢你的反馈（第一个反馈泪流满面）！
想了一下的确如你所说有这个美观问题。我按照你的建议改了一下，但是感觉还是不太满意，所以就直接调用fancyBox实现了这个功能……因为调用了jQuery和其他的库所以还不知道能不能审核通过，等到新版本审核通过了你再看看？
#5-5 - 2018-3-12 08:57
chitanda＠Lv2
后圣域传说 说: 非常感谢你的反馈（第一个反馈泪流满面）！
想了一下的确如你所说有这个美观问题。我按照你的建议改了一下，但是感觉还是不太满意，所以就直接调用fancyBox实现了这个功能……因为调用了jQuery和其他...
Fancybox的效果倒是不需要质疑。。不过这里加的库有点多不知道能不能过审了。。
#5-6 - 2018-3-12 16:05
Sai🖖
后圣域传说 说: 非常感谢你的反馈（第一个反馈泪流满面）！
想了一下的确如你所说有这个美观问题。我按照你的建议改了一下，但是感觉还是不太满意，所以就直接调用fancyBox实现了这个功能……因为调用了jQuery和其他...
现在这样会覆盖图片原有的链接吧
#5-7 - 2018-3-12 21:26
后圣域传说
Sai 说: 现在这样会覆盖图片原有的链接吧
并不会覆盖图片原有的链接，我只是添加了data-fancybox和href两个属性，原有的src属性还是保留了的。
#5-8 - 2018-3-12 21:36
Sai🖖
后圣域传说 说: 并不会覆盖图片原有的链接，我只是添加了data-fancybox和href两个属性，原有的src属性还是保留了的。
图片是可以加 url bbcode 的
#5-9 - 2018-3-12 22:10
后圣域传说
Sai 说: 图片是可以加 url bbcode 的
但是我查看了BBCode 标签指南，里面并没有说如何给图片添加网站链接，试了试并不能这样嵌套加BBCode，能否说明一下如何同时插入一张图片的时候带有url？（平时也没有见过谁这么做好像
不对，我这个例子好像就是可以的……等等我去改一下代码……
#5-10 - 2018-3-12 23:07
后圣域传说
Sai 说: 图片是可以加 url bbcode 的
我多加了一个判断，如果插入的图片已经有href的话就不把它放到fancyBox里面了。不过我觉得99%插入的图片都不会带url的BBCode吧……
#5-11 - 2018-3-13 01:52
灰灰
后圣域传说 说: 我多加了一个判断，如果插入的图片已经有href的话就不把它放到fancyBox里面了。不过我觉得99%插入的图片都不会带url的BBCode吧……
我的想法是用圖片的實際寬度與討論頁面寬度/圖片寬度相比較
比頁面寬才交給fancybox，而且建議給一個Alt提示，點擊會彈出fancybox什麼的

P.S. 圖片實際寬度可以使用 HTMLImageElement.naturalWidth 來獲取
#5-12 - 2018-3-13 02:01
后圣域传说
灰灰 说: 我的想法是用圖片的寬度與討論頁面寬度相比較
比頁面寬才交給fancybox，而且建議給一個Alt提示，點擊會彈出fancybox什麼的

P.S. 圖片寬度可以使用 HTMLImageElement....
之前加判断是为了处理有些图片自带href，而fancyBox是通过href来工作的，所以我排除了这些图片，实际Bangumi很少有人插入图片的同时在同一张图片上插入链接。
至于判断宽度，我倒是觉得统一一点比较好，不管多宽点击都可以用fancyBox。
给一个提示倒是个好想法，不过那应该是title而不是alt吧，我去试着加一下，也许可以提示一下图片本身带有href，可以访问什么的。
#5-13 - 2018-3-13 03:38
后圣域传说
Sai 说: 图片是可以加 url bbcode 的
我又修改了一下，现在应该fancyBox和url的BBCode不会冲突了，如果一张图片加了url的BBCode，点击图片会同时打开fancyBox和url。
#5-14 - 2018-3-15 14:52
后圣域传说
chitanda@Lv2 说: Fancybox的效果倒是不需要质疑。。不过这里加的库有点多不知道能不能过审了。。
新版本通过啦！快来试试看怎么样？
#5-15 - 2018-3-15 14:52
后圣域传说
小鸠世界第一可爱 说: +1，目前这个版本还是不太实用
来试试看新版本是否实用？
#5-16 - 2018-3-15 14:53
后圣域传说
灰灰 说: 我的想法是用圖片的實際寬度與討論頁面寬度/圖片寬度相比較
比頁面寬才交給fancybox，而且建議給一個Alt提示，點擊會彈出fancybox什麼的

P.S. 圖片實際寬度可以使用 HTMLImag...
新版本通过了，也已经添加上提示了！
#5-17 - 2018-3-15 15:09
灰灰
后圣域传说 说: 新版本通过了，也已经添加上提示了！
好耶ヽ(✿ﾟ▽ﾟ)ノ
有一点，建议使用
jQuery.ajax({

  dataType: "script",

  cache: true,

  url: url

});

来替代 $.getScript ，getScript是默认不缓存的
我的流量啊啊啊(bgm38)
#5-18 - 2018-3-15 15:42
Thunder
后圣域传说 说: 来试试看新版本是否实用？
新组件是仅对章节讨论有效吗？日志、讨论版、小组讨论的图片似乎没有反应bgm38
#5-19 - 2018-3-15 16:08
后圣域传说
灰灰 说: 好耶ヽ(✿ﾟ▽ﾟ)ノ
有一点，建议使用
jQuery.ajax({
  dataType: "script",
  cache: true,
 &nb...
嗯，其实之前考虑过这个问题，结果我发现组件里面的CSS也是不缓存每次都请求的，就没有管了……有没有什么好方法可以让外部的CSS文件也缓存呢？我现在是直接把整个外部的CSS文件放在组件的CSS里面，结果这个CSS文件挺大的不缓存感觉不太好。
#5-20 - 2018-3-15 16:12
后圣域传说
小鸠世界第一可爱 说: 新组件是仅对章节讨论有效吗？日志、讨论版、小组讨论的图片似乎没有反应bgm38
嗯，我现在只匹配了条目章节，因为这样子正则表达式写起来比较简单233，我明天就把这些加上。
#5-21 - 2018-3-15 16:32
chitanda＠Lv2
后圣域传说 说: 嗯，其实之前考虑过这个问题，结果我发现组件里面的CSS也是不缓存每次都请求的，就没有管了……有没有什么好方法可以让外部的CSS文件也缓存呢？我现在是直接把整个外部的CSS文件放在组件的CSS里面，结果...
讨论的显示效果没啥问题，不过有些原图本来就很小的我在想有没有必要调用fancybox。。
比如这里的
http://bgm.tv/ep/762292#post_564269
另外我电脑看了下css有缓存啊，没看懂你5-19那个是什么意思。。
#5-22 - 2018-3-15 19:45
灰灰
后圣域传说 说: 嗯，其实之前考虑过这个问题，结果我发现组件里面的CSS也是不缓存每次都请求的，就没有管了……有没有什么好方法可以让外部的CSS文件也缓存呢？我现在是直接把整个外部的CSS文件放在组件的CSS里面，结果...
#5-23 - 2018-3-15 22:18
后圣域传说
灰灰 说:

作为Firefox用户心态崩了……似乎在Firefox下组件的JavaScript和CSS都是不缓存的……这是什么情况？
#5-24 - 2018-3-15 23:09
灰灰
后圣域传说 说:
作为Firefox用户心态崩了……似乎在Firefox下组件的JavaScript和CSS都是不缓存的……这是什么情况？
嗯……那可以这样
$(document.head).append(`<link rel="stylesheet" href="https://cdn.jsdelivr.net/gh/fancyapps/[email protected]/dist/jquery.fancybox.min.css">`);
#5-25 - 2018-3-16 05:46
后圣域传说
小鸠世界第一可爱 说: 新组件是仅对章节讨论有效吗？日志、讨论版、小组讨论的图片似乎没有反应bgm38
1.2版本已经匹配了章节讨论、日志、讨论版、小组讨论，不知道还有没有什么遗漏。
#5-26 - 2018-3-16 05:47
后圣域传说
chitanda@Lv2 说: 讨论的显示效果没啥问题，不过有些原图本来就很小的我在想有没有必要调用fancybox。。
比如这里的
http://bgm.tv/ep/762292#post_564269
另外我电脑看了下css有缓...
嗯，不过我觉得为了显示效果的统一还是一起fancyBox了吧。
5-19的问题只有Firefox下有，Chrome下没有，我可能是Firefox的受害者……
#5-27 - 2018-3-16 05:50
后圣域传说
灰灰 说: 嗯……那可以这样
$(document.head).append(`<link rel="stylesheet" href="https://cdn.jsdelivr.net/gh/fancyapps...
嗯，总之不管怎么样在Chrome下都是会缓存组件的JavaScript和CSS文件的，不知道为啥Firefox不缓存。我按照你的思路改了一下，现在1.2版本至少Firefox和Chrome都会缓存fancyBox的相关文件了，至于Firefox下组件的相关文件不缓存我也懒得管了……
#5-28 - 2018-3-16 11:11
Sai🖖
后圣域传说 说: 嗯，总之不管怎么样在Chrome下都是会缓存组件的JavaScript和CSS文件的，不知道为啥Firefox不缓存。我按照你的思路改了一下，现在1.2版本至少Firefox和Chrome都会缓存fa...
现在这样设 cache 会影响全局缓存设置，用 $.ajax 直接取脚本吧。
#5-29 - 2018-3-16 11:32
后圣域传说
Sai 说: 现在这样设 cache 会影响全局缓存设置，用 $.ajax 直接取脚本吧。
嗯，我之前改得太急了，$.ajaxSetup()其实是不recommend的。现在用$.ajax()了。
#6 - 2018-4-1 02:55
灰灰 (もうぅ！プラチナむかつく！)
希望可以给回复后加载的图片也加上fancyBox
(bgm61)
#6-1 - 2018-4-1 23:08
后圣域传说
我更新了1.3版本，现在应该对回复后加载的图片也有效果了，不过我感觉我的实现方式不是很优雅，不知道有没有更好的写法……
#6-2 - 2018-4-2 02:40
灰灰
后圣域传说 说: 我更新了1.3版本，现在应该对回复后加载的图片也有效果了，不过我感觉我的实现方式不是很优雅，不知道有没有更好的写法……
用jQuery.ajaxComplete啦(bgm38)
能用就行(bgm38)
#6-3 - 2018-4-5 05:07
后圣域传说
灰灰 说: 用jQuery.ajaxComplete啦
能用就行
呃，其实我的问题是怎么样才能准确只选取到回复后的图片，我看刚回复的和之前回复的属性并没有什么差别，全部选取重复操作效率有点低，总感觉我现在用计数器来选取有点蠢？但是用Filter的话也还是得需要一个计数器来确定之前页面加载完成以后处理到哪个了吧？
我好像犯二了，直接选取最后一个回复然后处理里面的图片应该就可以了……
#6-4 - 2018-4-5 05:53
灰灰
后圣域传说 说: 呃，其实我的问题是怎么样才能准确只选取到回复后的图片，我看刚回复的和之前回复的属性并没有什么差别，全部选取重复操作效率有点低，总感觉我现在用计数器来选取有点蠢？但是用Filter的话也还是得需要一个计...
不需要计数器啦(bgm38)


监听ajaxComplete事件，拿到postid，处理新加的部分
#6-5 - 2018-4-6 16:56
后圣域传说
灰灰 说: 不需要计数器啦


监听ajaxComplete事件，拿到postid，处理新加的部分
搞定了，一开始上来直接用ajaxComplete，结果一堆因为注释所以没法JSON.parse的responseText的错误，后来才发现组件也都是AJAX加载的……