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

Manual Recordings: add optional UI element to set subtitle #646

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

Conversation

ulmus-scott
Copy link
Contributor

@ulmus-scott ulmus-scott commented Sep 30, 2022

If the element is absent, the behavior is unchanged.

I don't know how to test default and default-wide, but the changes are identical to the other themes.

Should this also bump the theme minor versions?

Checklist

@ulmus-scott
Copy link
Contributor Author

For earlier versions of MythTV, the theme changes for other themes will result in a textedit widget that does not effect the resultant recording rule, which may be confusing. However, this is still technically backwards compatible.

If the element is absent, the behavior is unchanged.
Move "title" up slightly and squeeze in "subtitle", moving the rest
down slightly, except "duration", which I moved up slightly to make
the spacing more even.
Move "title" up and squeeze in "subtitle", moving the rest down
slightly, except "duration", which I didn't move to make the
spacing more even.
Move "title" up and squeeze in "subtitle", moving the rest down
slightly, except "duration", which I didn't move to make the
spacing more even.
Move "title" up and squeeze in "subtitle", moving the rest down
slightly, except "duration", which I didn't move to make the
spacing more even.
Move "title" up and squeeze in "subtitle", moving the rest down
slightly, except "duration", which I didn't move to make the
spacing more even.
so it doesn't encroach on the new position of "title" and "titlelabel".
The position was copied from MythCenter-wide.  MythCenter and default
use the slightly different position of 20,5 instead.
@bennettpeter
Copy link
Member

The same change should be made in the web app. The web app has all the capabilities of the frontend for scheduling.

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.

None yet

2 participants