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

Overlay anchor points #253

Closed
wants to merge 1 commit into from
Closed

Overlay anchor points #253

wants to merge 1 commit into from

Conversation

3rob3
Copy link
Collaborator

@3rob3 3rob3 commented Jan 10, 2025

Allow user to set anchor points for clock/weather, image info, and web calendar.
Closes #194, Closes #250

@3rob3 3rob3 requested a review from JW-CH January 10, 2025 14:12
@3rob3 3rob3 added the enhancement New feature or request label Jan 10, 2025
@@ -38,6 +38,18 @@ private Task<IEnumerable<Guid>> ExcludedAlbumAssets
}
}

private Task<IEnumerable<Guid>>? _excludedPeopleAssets;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this in this PR? If you create new branches, make sure you branch from main, not from another branch

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, my bad. I switched back to Main first but must not have pulled.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed it for you :) you have to rebase to this branch again.

Webcalendars=https://calendar.mycalendar.com/basic.ics,webcal://calendar.mycalendar.com/basic.ics
WebCalendarPosition="top-0 right-0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Defaults should be something like this: bottom-[0px] because bottom-0 is a preset and not every number is supported. For example, bottom-30 will not work and should be bottom-[30px] instead. bottom-[30%] is also allowed. Maybe we should document somewhere, that all tailwind syntax is valid. Also maybe this should not be explicitly named XYPosition and instead could be custom classes in general for WebCalendar but then it probably would make more sense just to have a separate CSS to overwrite this stuff. What do you think @3rob3

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree. I originally had a custom position class to convert "BottomLeft" to a CSS compatible string. It got very confusing and I had a lot of trouble getting it to work. Then I thought it would also be nice to adjust from 0 so did it like I did. I couldn't get any value other than 0 to work right so your explanation makes sense.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still doesn't work right, i think because it's parent is not relative? IDK

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, i don't get it. When I put in any value other than zero it gets shoved all the way over to the left or top.

@JW-CH JW-CH force-pushed the dev.3rob3.OverlayAnchorPoints branch from 211e371 to 86a75b4 Compare January 10, 2025 14:41
@3rob3 3rob3 closed this Jan 10, 2025
@3rob3 3rob3 deleted the dev.3rob3.OverlayAnchorPoints branch January 19, 2025 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants