{"action":"opened","issue":{"url":"https://api.github.com/repos/ezQuake/ezquake-source/issues/177","repository_url":"https://api.github.com/repos/ezQuake/ezquake-source","labels_url":"https://api.github.com/repos/ezQuake/ezquake-source/issues/177/labels{/name}","comments_url":"https://api.github.com/repos/ezQuake/ezquake-source/issues/177/comments","events_url":"https://api.github.com/repos/ezQuake/ezquake-source/issues/177/events","html_url":"https://github.com/ezQuake/ezquake-source/issues/177","id":182281547,"number":177,"title":"REQ: Re-enable working brights in qwtf","user":{"login":"FlyingCaboose","id":22772060,"avatar_url":"https://avatars.githubusercontent.com/u/22772060?v=3","gravatar_id":"","url":"https://api.github.com/users/FlyingCaboose","html_url":"https://github.com/FlyingCaboose","followers_url":"https://api.github.com/users/FlyingCaboose/followers","following_url":"https://api.github.com/users/FlyingCaboose/following{/other_user}","gists_url":"https://api.github.com/users/FlyingCaboose/gists{/gist_id}","starred_url":"https://api.github.com/users/FlyingCaboose/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/FlyingCaboose/subscriptions","organizations_url":"https://api.github.com/users/FlyingCaboose/orgs","repos_url":"https://api.github.com/users/FlyingCaboose/repos","events_url":"https://api.github.com/users/FlyingCaboose/events{/privacy}","received_events_url":"https://api.github.com/users/FlyingCaboose/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-10-11T14:37:54Z","updated_at":"2016-10-11T14:37:54Z","closed_at":null,"body":"I discussed this with both dimman and meag.\r\n\r\nIn gl_rmain.c\r\n\r\nif (clmodel->modhint == MOD_PLAYER || ent->renderfx & RF_PLAYERMODEL) {\r\nfbskins = bound(0, r_fullbrightSkins.value, r_refdef2.max_fbskins);\r\nif (fbskins == 1 && gl_fb_models.value == 1) {\r\nambientlight = shadelight = 4096;\r\nfull_light = true;\r\n}\r\nelse if (fbskins == 0) {\r\nambientlight = max(ambientlight, 8);\r\nshadelight = max(shadelight, 8);\r\nfull_light = false;\r\n\r\n}\r\nelse if (fbskins) {\r\nambientlight = max(ambientlight, 8 + fbskins * 120);\r\nshadelight = max(shadelight, 8 + fbskins * 120);\r\nfull_light = true;\r\n\r\nMost TF servers support fullbrightskins anyways and the recent builds are just disabling all brights on skins..including the red dot on the player gun as well as the unique brights on class skins.\r\n\r\n74.86.171.201:27502 - tastyspleen.net::Meg aTF Classics allows r_fullbrightskins \"1\" as well as practically all MegaTF servers. Nice One (1) - Team Fortress appears to be taken down, so I couldn't test it there. That was basically the only other active server. AGR doesn't allow r_fullbrightskins, but it is an FTE driven server. Meaning that most players are going to see regular brights on player models regardless.\r\n\r\nAGR encourages the use of the FTE client. It utilizes stuff like csqc and such. So thus the majority of those players are going to see the original brights that have been disabled on ezQuake.\r\n\r\nI wanted to request this a long time ago, but I just got around to it."}}
{"actor":{"display_login":"FlyingCaboose"}}