Time Nick Message 13:05 MTDiscord I'd like to merge #13836 for 5.8.0 to avoid the confusion described in the PR with the new settings GUI on Android. 13:05 ShadowBot https://github.com/minetest/minetest/issues/13836 -- Android: Add `field_enter_after_edit[]` formspec element by grorp 13:06 MTDiscord It has the necessary approval, so I guess I'll merge it tomorrow if there are no objections. 13:06 sfan5 can we declare that element as "subject to removal at any time" in the docs? 13:07 sfan5 so we're not forced to support something that's just a workaround for mobile platforms in the future 13:07 sfan5 (assuming a better solution is found) 13:10 [MTMatrix] Can #13282 be added for 5.8 as well? See my latest comment 13:10 ShadowBot https://github.com/minetest/minetest/issues/13282 -- Remove controls listed in the pause menu (no touchscreen) by Zughy 13:10 [MTMatrix] (the fact that we can't see the bot messages on Matrix anymore is annoying, every time I have to find the issue/PR manually) 13:11 MTDiscord sfan5: I'll do that, but I'm not sure if there is a better solution. 13:19 rubenwardy I feel like that element could apply to desktop too - submitting when the field loses focus 13:20 rubenwardy or when it stops changing for x seconds 13:23 rubenwardy For #13282 to be merged, I'd like to see "Change Keys" renamed to "Controls" 13:23 ShadowBot https://github.com/minetest/minetest/issues/13282 -- Remove controls listed in the pause menu (no touchscreen) by Zughy 13:23 MTDiscord I'll write "subject to change or removal" then. 13:23 rubenwardy the change keys dialog does need a redesign very soon... 13:24 rubenwardy "Experimental: subject to change or removal" 13:24 rubenwardy experimental is a key term used a lot in the docs already 13:31 rubenwardy > I support doing this in the long term, can rename "Change Keys" to "Controls" and show a better interface there. As to whether this is a good idea with Change Keys in its current state is another matter 15:40 MTDiscord What about my 3dline PR? Does it have a chance to be re-reviewed/approved by anyone still before the 5.8 feature freeze? 16:28 Desour merging #13854 in 10 16:28 ShadowBot https://github.com/minetest/minetest/issues/13854 -- [NOSQUASH] Split up the internal sound stuff into src/client/sound and add a namespace by Desour 16:49 MTDiscord Andrey01: I don't think so. 18:02 MTDiscord Sadly 18:09 MTDiscord 5.8 could get a still one big feature as there is the only major one in the milestone which is the setting menu redesign and the remain ones are more minor (settings icon button, texture modifiers and physics overrides). 18:13 MTDiscord And my PR has already been ready long ago and reviewed many times by various members, for now all what necessary is to do a still quick surficial review and mark a still approval and that's all. 18:16 MTDiscord In any case further improvements will come only through next PRs, I don't intend to add anything anymore here as it is already large 18:29 rubenwardy I wonder if there's any interest in participating in hacktoberfest again? They have done a fair amount that should combat the spam, the problem is going to be how beginner friendly we are... issues in "good first issue" are a bit suspect tbh 18:30 rubenwardy andrey2470t: I haven't reviewed it as I'd like to see a review from someone that actually knows graphics well, like x2048 20:16 [MTMatrix] rubenwardy: considering there are 1.1k issues, it'd be nice I guess 20:22 [MTMatrix] andrey2470t: I think that something big like #13834 might benefit MT more than any other feature, even if I definitely understand your frustration 20:22 ShadowBot https://github.com/minetest/minetest/issues/13834 -- Batched rendering of particles by x2048 20:24 wsor4035 on that note, semi wondering if https://github.com/minetest/minetest/pull/13834#issuecomment-1741837031 was supposed to be a review, since it reads that way, but 🤷