This makes the independent execute commands (build-exec branch) configurable by the Set Build Commands dialog. I decided to use separate tabs for the build commands and the execute commands for clear grouping and to reduce dialog height and avoid adding translated strings:
![image](https://user-images.githubusercontent.com/1107820/67222034-5eeb7500-f424-11e9-822c-cd660c6db80f.png)
The first commit has the independent execute commands in a single Build tab as before for comparison. There also some small fixes to the independent execute code.
The one thing that needs fixing is that the filetype execute dialog rows show in grey and can't be reset - I've not got around to fixing that but thought it worth sharing my progress on the build-exec branch anyway.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2377
-- Commit Summary --
* [build commands dialog] Add rows for independent execute commands
* Make build_setup_dialog() append notebook page
* Fix initial build_items_count
* Append separate Build & Run tabs
* Don't vertically expand table rows
* Revert changes to translated group labels now we have tabs
* dialog fixes
* Reset row count for Execute tab
* Reduce `overloads` length; fix reading uninitialized `src`
-- File Changes --
M src/build.c (124)
M src/build.h (3)
M src/project.c (11)
-- Patch Links --
https://github.com/geany/geany/pull/2377.patchhttps://github.com/geany/geany/pull/2377.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2377
The "run" build commands don't work on my system.
the opened term said the run script was not executable.
Running the script manually (via `sh` as implemented in `build.c`) yields
```
oskar@torres ~ % /bin/sh -c /tmp/geany_run_script_HVQX9Z.sh
/bin/sh: /tmp/geany_run_script_HVQX9Z.sh: Permission denied
```
Which make sense since the file is not marked as executable:
```
oskar@torres ~ % ls -l /tmp/geany_run_script_HVQX9Z.sh
.rw-rw-r-- 256 oskar 19 Oct 19:23 /tmp/geany_run_script_HVQX9Z.sh
```
As can be seen in `build.c`, executable permissions are only set on apple platforms:
https://github.com/geany/geany/blob/75def8a24eb2dab8d376ae051806128827d34fb…
I changed this in my PR #2372 which solves the problem for me. I'm not sure if i'm missing something here.
I don't think passing the script through `sh` circumvents the need for execution rights but i'm not an expert on unix permissions so maybe someone else knows why this was expected to work and/or works for other people?
Thanks!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2374
Hello
For example, If I open a geany project with 2 documents.
- I click on "document » fold all"
- I click on "project » close"
No if I reopen the same project, "fold" status are lost, I need to click again on "document » fold all".
Please, can you save fold status?
Thanks (Geany 1.26)
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/793
(On my system) the "run" build commands didn't work (the opened term said the run script was not executable). For some reason the code here only made the run script executable on apple platforms. I removed this limitation to have the file chmod'ed on all unixoid platforms.
Works on my machine.
I'm very new to patches and PRs and such, lmk if i'm doing this wrong :)
Cheers
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2372
-- Commit Summary --
* Fix shell script generation in build.c
-- File Changes --
M src/build.c (3)
-- Patch Links --
https://github.com/geany/geany/pull/2372.patchhttps://github.com/geany/geany/pull/2372.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2372
It would be very useful to be able to search across multiple lines in "Find in Files". This doesn't work using patterns (\n) or by pasting in the line break character.
I realize this is all because `grep` doesn't really support multi-line searches, and it would not be difficult to implement this request.
It might be possible to add support for `pcregrep` to do what we need, with Geany complaining if you are searching for a linebreak-containing string and there being no `pcregrep` installed.
Or, we could dump grep and move find in files internally, using the same engine as the current Find feature.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2322
The existing popup menu lists all the notebook tabs first then shows command items below. This means the command items are harder to access (user has to move mouse further). They can also get pushed off screen when there are many tabs open (see https://github.com/geany/geany/pull/2346#issuecomment-539842133). Instead, let's split up the menu for better UX.
New notebook action widget menu, left or right click ☰ button:
![image](https://user-images.githubusercontent.com/1107820/66484161-63b43e80-ea9e-11e9-9d58-b6efef850baf.png)
Commands-only tab menu:
![image](https://user-images.githubusercontent.com/1107820/66500598-b6e7ba80-eab9-11e9-8908-3fb43bed2bdb.png)
The single close item has been grouped separately and put first because:
> This will be the most common choice for anyone who disables notebook tabs.
This also groups the commands closing multiple documents separately from the other commands. Closing multiple documents is a more drastic action so should be grouped separately.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2348
-- Commit Summary --
* Use gtk_widget_show_all in show_tab_bar_popup_menu
* Remove unnecessary casts
* Notebook tab menu: Move tab filenames to button menu at end of tabs
* Move single close item to top of menu
-- File Changes --
M src/notebook.c (56)
-- Patch Links --
https://github.com/geany/geany/pull/2348.patchhttps://github.com/geany/geany/pull/2348.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2348
Problem:
I am using geany 1.35 and while I can use my mouse to close other windows, I am unable to map any keybinding to Close Other. There is no option from what I can tell, it is just not mapped.
Background:
I have that keybinding exposed in other IDEs I use such as Intellij and so I am accustomed to using it to organize my workspace efficiently.
Proposed Solution:
Expose Close Other as an action that can have a keybinding mapped, optionally default to CTRL+SHIFT+O.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2368