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

fix:removed fix width for superadmin #1173

Merged
merged 2 commits into from
Dec 24, 2023
Merged

fix:removed fix width for superadmin #1173

merged 2 commits into from
Dec 24, 2023

Conversation

Vayras
Copy link
Contributor

@Vayras Vayras commented Dec 22, 2023

With this pull request the entire ui for superadmin frontend will scale to the screen , previously was set to fixed width

@@ -10,7 +10,7 @@ import AdminAccessDenied from './accessDenied';
const Container = styled.body`
height: 100vh; /* Set a fixed height for the container */
overflow-y: auto; /* Enable vertical scrolling */
width: 1366px;
/* width: 1366px; */
Copy link
Contributor

Choose a reason for hiding this comment

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

just delete it instead of commenting it out

removed the comment
@ecurrencyhodler
Copy link
Contributor

Some context for this pr.

#1085 (comment)

@Vayras
Copy link
Contributor Author

Vayras commented Dec 22, 2023

@ecurrencyhodler With this PR , as per the designer and as mentioned I have removed the fixed width for the bounties superadmin page

@kevkevinpal
Copy link
Contributor

lgtm @ecurrencyhodler we should think about a mobile view for the admin page since it looks like this after a certain breakpoint
Screenshot 2023-12-23 at 9 59 24 PM

And like this on mobile

Screenshot 2023-12-23 at 9 59 46 PM

@kevkevinpal kevkevinpal merged commit 986a1d6 into stakwork:master Dec 24, 2023
5 checks passed
elraphty pushed a commit that referenced this pull request Jan 26, 2024
* fix:removed fix width for superadmin

* removed fixed width 

removed the comment
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.

3 participants