-
-
Notifications
You must be signed in to change notification settings - Fork 313
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add "Add Layer" button to Toolbox #3282
base: master
Are you sure you want to change the base?
feat: add "Add Layer" button to Toolbox #3282
Conversation
This commit addresses the issue synfig#1139, where users were not noticing the "Add Layer" button in the "Layers Panel." The "Add Layer" button has been added to the Toolbox for better visibility. Fixes synfig#1139
Updated the title to match style/conventional commits style. @neesaaa, you can check https://www.conventionalcommits.org/en/v1.0.0/ to learn more about it. |
In response to collaborator @ice0's feedback on issue synfig#1139. Fix synfig#1139
can u please help me where to lookup as the code is enormous |
I like this improvement |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can u please help me where to lookup as the code is enormous
Probably the best place is in Dock_Toolbox::refresh()
.
Take a look at the end of Dock_Toolbox::update_tools
to see how it is done for the tools.
making the proper edits suggested from rodolforg and ice0 fix#1139
done hope it works well now |
butter descriptive name for the function ,and erased the empty lines fix synfig#1139
Honestly, I don't think this feature makes sense. If a person is using Synfig studio, then we can safely assume the person knows GIMP as well. And if the person is familiar with GIMP layers system, then he can automatically understand how Synfig layers system works. |
hi @yoSachinkr ,there are 4 ways to force quit any program however that does not make the quit button doesn't make sense ---------------------------------- |
Hi @neesaaa , I was actually talking about this current feature request you've worked on. |
Hello @yoSachinkr, I believe what he meant was that although there could be many ways of doing the same thing, it doesn't necessarily mean that they're all redundant and one is enough. |
Hi @neesaaa @mohamedAdhamc ! I get that having different options to do the same thing can be useful, but I've never seen a program that shows the same gui element in two places at once, like having two buttons to quit in the same window. That would just make the interface look messy. |
hi @yoSachinkr you clarified your point in two comments in the same window , which proves my point |
Hi @neesaaa All I'm saying is putting two gui elements in a window that do exactly the same thing is a bad gui design. |
@ice0 @rodolforg i think the build failed with no reason of mine |
Hi! Yes, there is a strange issue with build sometimes. I have rerun the failed jobs. |
Gtk::RadioToolButton *tool_button = manage(new Gtk::RadioToolButton()); | ||
tool_button->set_group(radio_tool_button_group); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it need to be a RadioToolButton
instead of a regular one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- sorry was having exams
- i made it like the already exist button in layer panel
- because too , to make all things unified as all toolbox buttons and their radio button group
- but if u have different opinion point to me where to search as from my search it can be many things
This commit addresses the issue #1139, where users were not noticing the "Add Layer" button in the "Layers Panel." The "Add Layer" button has been added to the Toolbox for better visibility.
Fix #1139