-
Notifications
You must be signed in to change notification settings - Fork 21
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
Make changes to datagrid #537
Conversation
guergana
commented
Sep 3, 2024
- fixes Changes to the datagrid #520
@pdelboca @romicolman the metadata panel now shows below the screen, i think before it was absolutely positioned. I am not sure if this belongs to this ticket. Maybe that was introduced when we made changes from flex to grid system? As you can see in the screenshot, now the user has to scroll to see the metadata panel. Before it was overlayed: |
@guergana we dicussed the issue with the metadata in the comments of this issue: #520 (comment) I think it would be nice to fix it here since we are implementing changes in the datagrid and the layout. Does it makes sense? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @guergana! I tested changes. Here are a few comments:
-
Just in case... are the font and size of the buttons Metadata, source, errors report, undo and redo and table cells aligned with the new design?
-
I know we removed tooltips from the datagrid and changes were implemented OK, but I can see them on the Publish and Save changes buttons now that we moved those features:
- I asked @faith to indicate if we will keep the red color (errors) as it is or if we need to adapt it (see comment on the issue). @Faithkenny can add a comment in the ticket so that Guergana can make changes if needed?
Hi @romicolman I tried to make a compromise between the buttons we currently have, as @pdelboca has been pointing out, and the design, since these buttons don't exist in the current code. I have removed the tooltips. :) |
Yes, sure @pdelboca |
Deploying opendataeditor with Cloudflare Pages
|
@romicolman @pdelboca the panel position is fixed now. @Faithkenny the color of the column with errors in the datagrid is Please review again everyone and let me know if further changes are needed. |
Hi @guergana! I have just tested changes:
If this is not a small change, please, let's work on it after the pre-release. |
It's done, please check again. |
I'm seeing the same color. Maybe the problem is connected to Mac? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked it again and I can see the new color on the datagrid