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

Hide the 5 bottom-tray tab buttons from ribbon #841

Open
whyboris opened this issue Jan 29, 2024 · 7 comments
Open

Hide the 5 bottom-tray tab buttons from ribbon #841

whyboris opened this issue Jan 29, 2024 · 7 comments
Labels
good first issue Good for newcomers

Comments

@whyboris
Copy link
Owner

Since the bottom tray can be opened by clicking on the tabs that stick out on the bottom, we don't need the 5 associated buttons in the top ribbon (they take up space and are a silly way to access the bottom tray).

Hide the 5 buttons from showing up 🪓

@whyboris whyboris added the good first issue Good for newcomers label Jan 29, 2024
@kmaehlmann
Copy link

Can I work on this issue?

@whyboris
Copy link
Owner Author

@kmaehlmann thank you for the offer! 🙇‍♂️

To be clear, these are the five icons I have in mind:

image

Please let me know if you have any questions or would like any pointers 🤝

@kmaehlmann
Copy link

kmaehlmann commented Feb 12, 2024

Hi, I am having trouble with the npm install for development. I can't exactly say what the problem is right now but I am having trouble following the steps to develop. I have node.js/npm installed but I am not sure what else to do. I am also not exactly sure where to start with this issue. Some help would be great, sorry I am new to most of this. Thank you.

@kmaehlmann
Copy link

I am mostly concerned with testing my changes, I am not sure how to do that.

@whyboris
Copy link
Owner Author

No worries at all - I'm happy to hear you're trying out coding 👨‍🚀

When you run npm install there may be an error (the word "ERROR" will likely appear somewhere in the output). To circumvent this issue, you can execute npm install --legacy-peer-deps in your terminal instead. Once that step finishes (there may be "WARNING" written somewhere, but there shouldn't be an "ERROR") you can run npm start. After this command (and about 1 minute) the app should show up.

If you can get to this stage - you are ready to start coding. Consider using VS Code and in the app open the Video-Hub-App folder - so you can browse around and edit files.

Once you make changes to any file (.ts files contain the code) and save, the running Video Hub App instance should reload - and you should be able to see changes.

Look around this file: settings-buttons.ts (link) and try removing (in your text editor / VS Code) showRelatedVideosTray from the line I linked to above, then save the file, and the app should reload with the button gone from the UI.

Unfortunately, just deleting the button like that will probably prevent it from showing up in the tray, so you may need to change some other things. Look for the showRelatedVideosTray string (text) in this file (use CTRL + F to find the text in the file) - and you'll likely find it around line 668. These are the properties of the button (direct link)

I don't remember right, but maybe, if we're lucky, there is a way to hide the button from the ribbon while still showing it in the settings. There are other buttons around that behave this way I think -- maybe if you see how they are configured (they are part of this settings-buttons.ts file) - you may be able to fix this Issue 👍

I'm happy to help more if you'd like 😁

@kmaehlmann
Copy link

Hi boris, thank you for all your help.
image
Removing these 4 lines seemed to have removed the top ribbon buttons.

@whyboris
Copy link
Owner Author

@kmaehlmann - I'm glad you were able to get code to work on your computer 🥳

I suspect now that the buttons are removed from the tray, they are no longer showing up in the settings menu. I suspect now with your code change there is no way to toggle them on / off from showing up in the bottom tray (they are either showing or hidden - and the user doesn't have the ability to show them / hide them in the bottom tray).

The main task of this Issue is to hide the buttons from the ribbon, but still show them in the settings.

Take a glance at the settings menu - there are some buttons that do not have an eye icon next to them (so they never appear in the ribbon) - perhaps in their configuration there is something that makes them not show up in the ribbon, but still show up in the settings menu 😉

lyss0306 added a commit to lyss0306/Video-Hub-App-SwEProj that referenced this issue Sep 13, 2024
Solution to issue whyboris#841
Removed 'showFreq', 'showRelatedVideosTray', 'showRecentlyPlayed', 'showDetailsTray' from group 6
Added 'showFreq', 'showRelatedVideosTray', 'showRecentlyPlayed', 'showDetailsTray' to group 14 (settings tab 3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants