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

Request to add option to fill page or make reactive #744

Open
NightHawkATL opened this issue Nov 27, 2024 · 2 comments
Open

Request to add option to fill page or make reactive #744

NightHawkATL opened this issue Nov 27, 2024 · 2 comments
Labels
a:feature New feature or request in:ui

Comments

@NightHawkATL
Copy link

Is your feature request related to a problem? Please describe.
I have a few notifications that are long and wide and I have to scroll to get to the horizontal scroll bar to see the wider notifications.
image

Describe the solution you'd like
It would be nice to have the notifications window fill the available area to allow for viewing the wider notifications either with an option to "fill" or have the notification area be reactive to the size of the screen or window viewing them.

Describe alternatives you've considered
not using Gotify as there are others that offer this but aren't as easy to configure if there is no native Gotify support.

Additional context
none at the moment as the above should suffice, but let me know if you have questions.

@NightHawkATL NightHawkATL added the a:feature New feature or request label Nov 27, 2024
@eternal-flame-AD
Copy link
Member

Thanks for the ticket, I think it is because of many by-pixel max width styles in the code:

messageContentWrapper: {
width: '100%',
maxWidth: 585,

const DefaultPage: FC<IProps> = ({title, rightControl, maxWidth = 700, children}) => (
<main style={{margin: '0 auto', maxWidth}}>
<Grid container spacing={4}>
<Grid item xs={12} style={{display: 'flex', flexWrap: 'wrap'}}>
<Typography variant="h4" style={{flex: 1}}>
{title}
</Typography>
{rightControl}
</Grid>
{children}
</Grid>
</main>
);

I am not familiar with frontend but maybe this can become a relative constraint? @jmattheis

@jmattheis
Copy link
Member

Yeah, I think we should have some min-width so that small messages keep the current design and only bigger messages get more width. Maybe something like: min(100%, max(fit-to-content, 700px)).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:feature New feature or request in:ui
Development

No branches or pull requests

3 participants