Link label overwrites group label #18

Closed
opened 2019-07-12 22:05:56 +00:00 by flamingcow66 · 1 comment
flamingcow66 commented 2019-07-12 22:05:56 +00:00 (Migrated from github.com)

https://architype.io/#eyJ2ZXJzaW9uIjoxLCJnZW5lcmF0aW9uIjozMTMsIm5leHRJZCI6MjgyLCJlZGl0b3IiOlt7InR5cGUiOiJsaW5rIiwiaWQiOiJlbnRyeTE1MyIsImxhYmVsIjoiSFRUUFMiLCJmcm9tIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTE1NCIsImxhYmVsIjoiY2xvdWRmbGFyZSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJ0byI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxNTUiLCJsYWJlbCI6ImZyb250ZW5kMSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJsaW5rIiwiaWQiOiJlbnRyeTE1NiIsImxhYmVsIjoiZ1JQQyIsImZyb20iOnsidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MTU3IiwibGFiZWwiOiJmcm9udGVuZDEiLCJoaWdobGlnaHQiOmZhbHNlfSwidG8iOnsidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MTU4IiwibGFiZWwiOiJiYWNrZW5kMSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJsaW5rIiwiaWQiOiJlbnRyeTI3OSIsImxhYmVsIjoiSFRUUFMiLCJmcm9tIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTI4MCIsImxhYmVsIjoiY2xvdWRmbGFyZSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJ0byI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkyODEiLCJsYWJlbCI6ImZyb250ZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9LCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJncm91cCIsImlkIjoiZW50cnkyMzciLCJsYWJlbCI6bnVsbCwibWVtYmVycyI6W3sidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MjM4IiwibGFiZWwiOiJmcm9udGVuZDEiLCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTIzOSIsImxhYmVsIjoiYmFja2VuZDEiLCJoaWdobGlnaHQiOmZhbHNlfV0sImhpZ2hsaWdodCI6ZmFsc2V9LHsidHlwZSI6ImxpbmsiLCJpZCI6ImVudHJ5MTc3IiwibGFiZWwiOiJnUlBDIiwiZnJvbSI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxNzgiLCJsYWJlbCI6ImZyb250ZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9LCJ0byI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxNzkiLCJsYWJlbCI6ImJhY2tlbmQyIiwiaGlnaGxpZ2h0IjpmYWxzZX0sImhpZ2hsaWdodCI6ZmFsc2V9LHsidHlwZSI6Imdyb3VwIiwiaWQiOiJlbnRyeTE4NyIsImxhYmVsIjoidXMtd2VzdDEiLCJtZW1iZXJzIjpbeyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxODgiLCJsYWJlbCI6ImZyb250ZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9LHsidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MTg5IiwibGFiZWwiOiJiYWNrZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9XSwiaGlnaGxpZ2h0IjpmYWxzZX0seyJ0eXBlIjoibGluayIsImlkIjoiZW50cnkxODEiLCJsYWJlbCI6bnVsbCwiZnJvbSI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxODIiLCJsYWJlbCI6ImJhY2tlbmQxIiwiaGlnaGxpZ2h0IjpmYWxzZX0sInRvIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTE4MyIsImxhYmVsIjoiZGF0YWJhc2UiLCJoaWdobGlnaHQiOmZhbHNlfSwiaGlnaGxpZ2h0IjpmYWxzZX0seyJ0eXBlIjoibGluayIsImlkIjoiZW50cnkxODQiLCJsYWJlbCI6bnVsbCwiZnJvbSI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxODUiLCJsYWJlbCI6ImJhY2tlbmQyIiwiaGlnaGxpZ2h0IjpmYWxzZX0sInRvIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTE4NiIsImxhYmVsIjoiZGF0YWJhc2UiLCJoaWdobGlnaHQiOmZhbHNlfSwiaGlnaGxpZ2h0IjpmYWxzZX1dLCJzZWxlY3RlZCI6ImVudHJ5MjgyIn0=

https://architype.io/#eyJ2ZXJzaW9uIjoxLCJnZW5lcmF0aW9uIjozMTMsIm5leHRJZCI6MjgyLCJlZGl0b3IiOlt7InR5cGUiOiJsaW5rIiwiaWQiOiJlbnRyeTE1MyIsImxhYmVsIjoiSFRUUFMiLCJmcm9tIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTE1NCIsImxhYmVsIjoiY2xvdWRmbGFyZSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJ0byI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxNTUiLCJsYWJlbCI6ImZyb250ZW5kMSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJsaW5rIiwiaWQiOiJlbnRyeTE1NiIsImxhYmVsIjoiZ1JQQyIsImZyb20iOnsidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MTU3IiwibGFiZWwiOiJmcm9udGVuZDEiLCJoaWdobGlnaHQiOmZhbHNlfSwidG8iOnsidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MTU4IiwibGFiZWwiOiJiYWNrZW5kMSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJsaW5rIiwiaWQiOiJlbnRyeTI3OSIsImxhYmVsIjoiSFRUUFMiLCJmcm9tIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTI4MCIsImxhYmVsIjoiY2xvdWRmbGFyZSIsImhpZ2hsaWdodCI6ZmFsc2V9LCJ0byI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkyODEiLCJsYWJlbCI6ImZyb250ZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9LCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJncm91cCIsImlkIjoiZW50cnkyMzciLCJsYWJlbCI6bnVsbCwibWVtYmVycyI6W3sidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MjM4IiwibGFiZWwiOiJmcm9udGVuZDEiLCJoaWdobGlnaHQiOmZhbHNlfSx7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTIzOSIsImxhYmVsIjoiYmFja2VuZDEiLCJoaWdobGlnaHQiOmZhbHNlfV0sImhpZ2hsaWdodCI6ZmFsc2V9LHsidHlwZSI6ImxpbmsiLCJpZCI6ImVudHJ5MTc3IiwibGFiZWwiOiJnUlBDIiwiZnJvbSI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxNzgiLCJsYWJlbCI6ImZyb250ZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9LCJ0byI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxNzkiLCJsYWJlbCI6ImJhY2tlbmQyIiwiaGlnaGxpZ2h0IjpmYWxzZX0sImhpZ2hsaWdodCI6ZmFsc2V9LHsidHlwZSI6Imdyb3VwIiwiaWQiOiJlbnRyeTE4NyIsImxhYmVsIjoidXMtd2VzdDEiLCJtZW1iZXJzIjpbeyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxODgiLCJsYWJlbCI6ImZyb250ZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9LHsidHlwZSI6Im5vZGUiLCJpZCI6ImVudHJ5MTg5IiwibGFiZWwiOiJiYWNrZW5kMiIsImhpZ2hsaWdodCI6ZmFsc2V9XSwiaGlnaGxpZ2h0IjpmYWxzZX0seyJ0eXBlIjoibGluayIsImlkIjoiZW50cnkxODEiLCJsYWJlbCI6bnVsbCwiZnJvbSI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxODIiLCJsYWJlbCI6ImJhY2tlbmQxIiwiaGlnaGxpZ2h0IjpmYWxzZX0sInRvIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTE4MyIsImxhYmVsIjoiZGF0YWJhc2UiLCJoaWdobGlnaHQiOmZhbHNlfSwiaGlnaGxpZ2h0IjpmYWxzZX0seyJ0eXBlIjoibGluayIsImlkIjoiZW50cnkxODQiLCJsYWJlbCI6bnVsbCwiZnJvbSI6eyJ0eXBlIjoibm9kZSIsImlkIjoiZW50cnkxODUiLCJsYWJlbCI6ImJhY2tlbmQyIiwiaGlnaGxpZ2h0IjpmYWxzZX0sInRvIjp7InR5cGUiOiJub2RlIiwiaWQiOiJlbnRyeTE4NiIsImxhYmVsIjoiZGF0YWJhc2UiLCJoaWdobGlnaHQiOmZhbHNlfSwiaGlnaGxpZ2h0IjpmYWxzZX1dLCJzZWxlY3RlZCI6ImVudHJ5MjgyIn0=
flamingcow66 commented 2019-07-13 03:58:48 +00:00 (Migrated from github.com)

91c106a isn't a complete fix, but it fixes the test case and adds two separate heuristics (one for lines and one for labels). If we find this again, we may just need to ban line labels over group titles, the way we already ban them over nodes. That's tricky, however, because we don't want to ban them over groups. Please reopen with repro.

91c106a isn't a complete fix, but it fixes the test case and adds two separate heuristics (one for lines and one for labels). If we find this again, we may just need to ban line labels over group titles, the way we already ban them over nodes. That's tricky, however, because we don't want to ban them over groups. Please reopen with repro.
Sign in to join this conversation.