<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=793526" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/b4n">@b4n</a> and other gtk spurts</p>
<p>The code to detect being on the tab bar background uses styles "scroll-arrow-hlength" and "scroll-arrow-vlength" as part of the calculation of position.</p>
<p>But those appear to be scrollbar arrows, not notebook tabs arrows <a href="https://developer.gnome.org/gtk3/stable/GtkWidget.html" rel="nofollow">here</a> and are not mentioned in <a href="https://developer.gnome.org/gtk3/stable/GtkNotebook.html" rel="nofollow">notebook styles</a>.  Its theme dependent how big scrollbar arrows and tabbar arrows are, so maybe where scrollbar arrows are small and tabbar arrows are large <a href="https://github.com/geany/geany/blob/97547edb52e34b8f78f44a13548a0fb55baf1a44/src/notebook.c#L351">this</a> code is finding parts of the tabbar arrow are outside the arrow and ripe for double clicking?</p>
<p>IMHO that code and the widget layout dependent checks here are wrong, they depend on implementation dependent things and now some implementations have changed.</p>
<p>Since the feature is implemented wrongly and is undiscoverable it should indeed be removed (by commenting if you want) until "somebody" PRs a correct implementation that doesn't depend on styles and GTK widget layouts.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/issues/1890#issuecomment-431709235">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ5UmzbUh4tAWFzM9im9Octfi3tAAks5unPMGgaJpZM4VCA7p">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABDrJ6WfLI7MdjhMxYQx_mWFA4tyRHerks5unPMGgaJpZM4VCA7p.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@elextr in #1890: @b4n and other gtk spurts\r\n\r\nThe code to detect being on the tab bar background uses styles \"scroll-arrow-hlength\" and \"scroll-arrow-vlength\" as part of the calculation of position.\r\n\r\nBut those appear to be scrollbar arrows, not notebook tabs arrows [here](https://developer.gnome.org/gtk3/stable/GtkWidget.html) and are not mentioned in [notebook styles](https://developer.gnome.org/gtk3/stable/GtkNotebook.html).  Its theme dependent how big scrollbar arrows and tabbar arrows are, so maybe where scrollbar arrows are small and tabbar arrows are large [this](https://github.com/geany/geany/blob/97547edb52e34b8f78f44a13548a0fb55baf1a44/src/notebook.c#L351) code is finding parts of the tabbar arrow are outside the arrow and ripe for double clicking?\r\n\r\nIMHO that code and the widget layout dependent checks [here](ttps://github.com/geany/geany/blob/97547edb52e34b8f78f44a13548a0fb55baf1a44/src/notebook.c#L517) are wrong, they depend on implementation dependent things and now some implementations have changed.\r\n\r\nSince the feature is implemented wrongly and is undiscoverable it should indeed be removed (by commenting if you want) until \"somebody\" PRs a correct implementation that doesn't depend on styles and GTK widget layouts."}],"action":{"name":"View Issue","url":"https://github.com/geany/geany/issues/1890#issuecomment-431709235"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/issues/1890#issuecomment-431709235",
"url": "https://github.com/geany/geany/issues/1890#issuecomment-431709235",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [geany/geany] switch between document tab when clicking on arrow button bug (#1890)",
"sections": [
{
"text": "",
"activityTitle": "**elextr**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@elextr",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"geany/geany\",\n\"issueId\": 1890,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"geany/geany\",\n\"issueId\": 1890\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/geany/geany/issues/1890#issuecomment-431709235"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 352849641\n}"
}
],
"themeColor": "26292E"
}
]</script>