Skip to content
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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

neesaaa
Copy link
Contributor

@neesaaa neesaaa commented Nov 26, 2023

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

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
@mohamedAdhamc mohamedAdhamc changed the title Add "Add Layer" button to Toolbox feat: add "Add Layer" button to Toolbox Nov 26, 2023
@mohamedAdhamc
Copy link
Contributor

mohamedAdhamc commented Nov 26, 2023

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.

@ice0
Copy link
Collaborator

ice0 commented Dec 4, 2023

Hi, guys!

Looks good, the only problem I found is that if I close all windows this button is still active.
It might be worth disabling it like all other buttons.

image

@neesaaa
Copy link
Contributor Author

neesaaa commented Dec 7, 2023

can u please help me where to lookup as the code is enormous

@morevnaproject
Copy link
Member

I like this improvement

Copy link
Contributor

@rodolforg rodolforg left a 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.

synfig-studio/src/gui/docks/dock_toolbox.cpp Outdated Show resolved Hide resolved
synfig-studio/src/gui/docks/dock_toolbox.cpp Outdated Show resolved Hide resolved
synfig-studio/src/gui/docks/dock_toolbox.cpp Outdated Show resolved Hide resolved
    making the proper edits suggested from rodolforg and ice0
fix#1139
@neesaaa
Copy link
Contributor Author

neesaaa commented Dec 9, 2023

done hope it works well now

    butter descriptive name for the function ,and erased the empty lines
fix synfig#1139
@yoSachinkr
Copy link

yoSachinkr commented Dec 10, 2023

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.
There are already too many ways to add layers in Synfig. From the context menu, from the menu bar, from the context menu at layers panel, from the toolbar of layers panel.
I don't see the need of adding one more button just to add layers.

@neesaaa
Copy link
Contributor Author

neesaaa commented Dec 10, 2023

hi @yoSachinkr ,there are 4 ways to force quit any program however that does not make the quit button doesn't make sense

----------------------------------
1st- way Press Alt + F4
2nd-1. Press Control + Alt + Delete. 2. Choose Task Manager.3. Select the application that you want to force quit.4. Click End task.
3rd-1. Press Windows key + R.2. Type cmd into the search box and press Enter.3. Type tasklist into the Command Prompt. You'll then see a list of tasks and programs running on your computer.
4th- Enter taskkill/im [name_of_program].exe. For example, if you want to quit Firefox, you'd enter taskkill/im firefox.exe.Press Enter.
check :https://www.businessinsider.com/guides/tech/how-to-force-quit-on-windows

@yoSachinkr
Copy link

hi @yoSachinkr ,there are 4 ways to force quit any program however that does not make the quit button doesn't make sense

---------------------------------- 1st- way Press Alt + F4 2nd-1. Press Control + Alt + Delete. 2. Choose Task Manager.3. Select the application that you want to force quit.4. Click End task. 3rd-1. Press Windows key + R.2. Type cmd into the search box and press Enter.3. Type tasklist into the Command Prompt. You'll then see a list of tasks and programs running on your computer. 4th- Enter taskkill/im [name_of_program].exe. For example, if you want to quit Firefox, you'd enter taskkill/im firefox.exe.Press Enter. check :https://www.businessinsider.com/guides/tech/how-to-force-quit-on-windows

Hi @neesaaa , I was actually talking about this current feature request you've worked on.
All I meant was this "add layer" button in the toolbox is unnecessary.

@mohamedAdhamc
Copy link
Contributor

Hi @neesaaa , I was actually talking about this current feature request you've worked on. All I meant was this "add layer" button in the toolbox is unnecessary.

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.

@yoSachinkr
Copy link

there are 4 ways to force quit any program however that does not make the quit button doesn't make sense

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.
Also, we all know that the add layer button is not really a tool like the others. Putting it in the toolbox would make Synfig look like a jumbled mess.
If we really need another add layer button, it should go in the toolbar of the canvas panel.

@neesaaa
Copy link
Contributor Author

neesaaa commented Dec 16, 2023

there are 4 ways to force quit any program however that does not make the quit button doesn't make sense

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. Also, we all know that the add layer button is not really a tool like the others. Putting it in the toolbox would make Synfig look like a jumbled mess. If we really need another add layer button, it should go in the toolbar of the canvas panel.

hi @yoSachinkr you clarified your point in two comments in the same window , which proves my point

@yoSachinkr
Copy link

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.

@neesaaa
Copy link
Contributor Author

neesaaa commented Dec 19, 2023

@ice0 @rodolforg i think the build failed with no reason of mine

@ice0
Copy link
Collaborator

ice0 commented Dec 19, 2023

@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.

synfig-studio/src/gui/docks/dock_toolbox.cpp Outdated Show resolved Hide resolved
Comment on lines +241 to +242
Gtk::RadioToolButton *tool_button = manage(new Gtk::RadioToolButton());
tool_button->set_group(radio_tool_button_group);
Copy link
Contributor

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?

Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Add Layer" button to toolbar?
6 participants