"Edit layer groups" option not found

[nobody]

Pcb-20070208 page 7 - 2.4 - Layers: refers to the "Edit layer groups" option to be found in the Settings menu.

Sorry, not found

[Felix Ruoff (felixruoff)]

Documentation-bug is stil present in the actual git-version. The appended patch corrects the documentation.

[Ineiev (ineiev)]

This sencence was written for the Lesstif hid (there is Settings->"Edit layer groupings" option).

Can you rewrite the patch to take this into account (or should we make GTK and Lesstif HIDS match each other)?

[Felix Ruoff (felixruoff)]

Thanks for the feedback!
I have had a look about matching the behaviour of gtk to the lesstif-gui, but it's more difficult than just adding a menu-entry. For this reason, I modified the patch for the documentation.

[Ineiev (ineiev)]

Thank you; I see two chunks there. The first one looks reasonable (I've just make the terminology consistent
with the rest pcb.texi).

The second chunk changes the "Menus" subsection which seems to be too outdated (e.g. there are described
"Screen" and "Sizes" popup menus absent both in the lesstif and GTK+ GUI); the whole subsection
calls for thorough revision IMO, probably via another bug report.

What do you think?

[Felix Ruoff (felixruoff)]

Thanks for the reply and the modifications in the first chunk!

For the second chunk I would think: This section really needs a revision, which should be done in a separate bug-report. Anyway I would suggest to apply the second chunk of my patch (again with modifications to make the terminology consistent) because it improves the documentation a (very little) bit. The one, who will do this review, will eventually miss this difference in lesstif GUI and GTK+ GUI if this is not mentioned there.

I appendet a modificated patch of the second chunk (fixes lesstif-gui -> lesstif GUI and gtk-gui -> GTK+ GUI9.

[Ineiev (ineiev)]

I've pushed the first chunk.

Regarding the second one, I think that section, "Menus", should be regenerated from comments
in gpcb-menu.res and pcb-menu.res, like the actions list.

I'll try to suggest a patch in another bug report.