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

[BUG] 3.5+7435 Reordering pages breaks 'Set to page' buttons #3082

Open
2 tasks done
JStrauber opened this issue Oct 11, 2024 · 6 comments
Open
2 tasks done

[BUG] 3.5+7435 Reordering pages breaks 'Set to page' buttons #3082

JStrauber opened this issue Oct 11, 2024 · 6 comments
Labels
BUG Something isn't working
Milestone

Comments

@JStrauber
Copy link

Is this a bug in companion itself or a module?

  • I believe this to be a bug in companion

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

On the latest Beta of 3.5, testing working with the new Pages setup where you can reorder / add / remove pages. Noticing if I have buttons / pages that set to other pages, for example setting a table of contents to link to other pages, if the pages are reordered, the 'Set to page' command still references the same 'page number' and doesn't follow the desired page itself.

Steps To Reproduce

  1. Build a button with 'internal: Set to page' and set it for current surface, then another page listed
  2. Add a new page before the existing destination page or move the existing page
  3. The 'Set to Page' button still points to the previous 'page number' which may be a different page or blank page

Expected Behavior

  1. Build a button with 'internal: Set to page' and set it for current surface, then another page listed
  2. Add a new page before the existing destination page or move the existing page
  3. The 'set to page' destination follows the set page no matter what page number it is

Environment (please complete the following information)

  • OS: Windows 11
  • Browser: Microsoft Edge 129.0.2792.89
  • Companion Version: v3.5.0+7435-main-80a9c16e

Additional context

No response

@JStrauber JStrauber added the BUG Something isn't working label Oct 11, 2024
@Julusian
Copy link
Member

Julusian commented Oct 11, 2024

This is expected currently, and is unlikely anything we can fix with the current actions.
I do intend to add some new actions which reference a page id (instead of page number), and so will follow the page as it gets moved. But these haven't been done yet

While in theory we could update page numbers as you move things around, I think that it will be very easy to make a bigger mess.
Maybe you are reordering because you want to change what page 2 including with these actions.
Or you delete page 2, should those actions remain referencing page 2, or something else? If you then insert a new page 2, those actions could now be pointing to 3. Is that what should happen?

@JStrauber
Copy link
Author

JStrauber commented Oct 11, 2024 via email

@Julusian
Copy link
Member

@JStrauber replying by email doesn't support attaching images (it looks like you tried to add an image) and for some reason the formatting is all messed up too, its lost all the newlines..

@JStrauber
Copy link
Author

So in our case we’re doing a setup where we are going to add new pages vaguely frequently (thank you for the more than 99 part!), and as part of organization we’d like to keep everything grouped together rather than just adding to the end. What would help us in this case is to be able to ‘link’ the button to a specific page, so that way if we rearrange the pages, it doesn’t break all the links. For now, we know that we’ll have to update if we move things around, which is fine.

“If you then insert a new page 2, those actions could now be pointing to 3. Is that what should happen”
This functionality would be great, to your point I get that it might not be best for everyone. I wonder if at some point there could be a selection where you can choose to set to page ‘by number’ and set to page ‘by name’, or something along those lines. Like, default could be always follow the selected page number, but you could check a box so it would automatically follow the intended ‘page’ if it moves in the list order.

(See screenshot attached)

So in this case, currently, if I add another ‘Monitor Pg’ in this list, I would then have to go back and edit all other pages that reference every higher page. It would be great to have it so that you can have the buttons follow the ‘intended’ page and not just the number. I can see benefits to both ways, which is why I suggest some sort of toggle. My guess is being able to follow a specific ‘page’ would require each page to have a separate type of UID other than just the page number or name, especially if someone used duplicate names.

Image

@digitaldrummerj
Copy link

To get around the current behavior of not updating the pages when they move, I create a variable for each page that I want to use in the "Set Page To" action so that I only need to update a single variable when the page moves and all of the buttons get updated.

@davidjoshuaford
Copy link

I've been looking at the same issue -

I don't think moving pages should change the linked Actions for "Set surface to PAGE NUMBER", but it would be helpful to have an Action that could "Set Surface to PAGE NAME".

That way pages can be reordered however you like and the page name keeps the integrity of the surface jumps.

Would have to be unique page names though, so perhaps generating a new page would auto name it New Page 2, New Page 3 etc.

It would be helpful if these could accept custom variables to be able to jump to specific pages.

For example, I have seven AUDIO pages AUDIO1, AUDIO2, AUDIO3 etc. So an Action to set a Surface to a page name variable such as "AUDIO$(internal:custom_audio_page)" would help simplify which audio page to jump to.

Also a question about Expressions in the page name, would a name like "AUDIO-1" potentially be an issue with a minus operation? ie better to name it AUDIO_1?

@Julusian Julusian added this to the v3.5 milestone Oct 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Something isn't working
Projects
Status: No status
Development

No branches or pull requests

4 participants