<p>Having something like "Show sidebar" don't make super much sense when its something you can toggle.<br>
So it says "Show sidebar" both when its enabled and disabled.<br>
Its like having a light switch with the text "Turn on the light" regardless of the light being on or off.</p>
<p>If you really want to use "Show sidebar" then i guess it should say "Show sidebar" when the sidebar is hidden and "Hide sidebar" when the sidebar is showed.</p>
<p>But since we got a checkmark there is no need for "Show" and "Hide". The checkmark tells the user if its on or off.</p>
<p>When its in the View menu people know the menu items are about viewing something.</p>
<p>There is also some strings starting with "Show " in Preferences - Editor - View under View.<br>
There "Show " could also be removed because they are all under "View".</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/pull/1788#issuecomment-369385357">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJyogyphHryw7whRz9wJGCRM7h220ks5tZcH3gaJpZM4SWkZJ">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ7yD15nYkZxt1jnGDP9bJ20_jvdcks5tZcH3gaJpZM4SWkZJ.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1788#issuecomment-369385357"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<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://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@scootergrisen in #1788: Having something like \"Show sidebar\" don't make super much sense when its something you can toggle.\r\nSo it says \"Show sidebar\" both when its enabled and disabled.\r\nIts like having a light switch with the text \"Turn on the light\" regardless of the light being on or off.\r\n\r\nIf you really want to use \"Show sidebar\" then i guess it should say \"Show sidebar\" when the sidebar is hidden and \"Hide sidebar\" when the sidebar is showed.\r\n\r\nBut since we got a checkmark there is no need for \"Show\" and \"Hide\". The checkmark tells the user if its on or off.\r\n\r\nWhen its in the View menu people know the menu items are about viewing something.\r\n\r\nThere is also some strings starting with \"Show \" in Preferences - Editor - View under View.\r\nThere \"Show \" could also be removed because they are all under \"View\"."}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1788#issuecomment-369385357"}}}</script>