-
Notifications
You must be signed in to change notification settings - Fork 7
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
[Feature Request] Show operation could be easier to read #12
Comments
Of course... we could take this further:
|
I think your last proposal is the better, since I would like to also propose that we only show the diff attributes at first, and maybe add a |
Update: did the MUST and SHOULDs in #15 We can also do the COULD, yes. I don't see a lot of benefit in having a "Show unchanged" button. Since this page's whole reason to exist is to show the activity log... if the developer has chosen to log the unchanged... I think the unchanged should be shown. Highlighting is one thing... hiding content is another, if you ask me. 🤷♂️ |
Since most of the stuff here is done, I'm going to close this issue and open a new one about the COUD-DO - #20 |
Feature Request
What I am trying to achieve
See the changes in an entry, where I logged everything (not just the dirty). To rephrase, the activity log entry contains the information about the fields that have changed, but also about the fields that have NOT changed. So it's got a lot of info.
Right now the Show view looks like this:
![CleanShot 2023-08-29 at 10 50 26@2x](https://private-user-images.githubusercontent.com/1032474/263942157-c7653b9e-55d7-4f82-bd60-97cde828eee8.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkyMzIzOTQsIm5iZiI6MTczOTIzMjA5NCwicGF0aCI6Ii8xMDMyNDc0LzI2Mzk0MjE1Ny1jNzY1M2I5ZS01NWQ3LTRmODItYmQ2MC05N2NkZTgyOGVlZTgucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIxMSUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMTFUMDAwMTM0WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9NmNjNTAzNjYxZTFmYWYwYWI0NjI5MDI2MTZhM2UxNDA2MmFlMTBkYTNmMDAzZWRhYzRhNmUxZDc4NGYyYmJjNSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.gf7_kfG_RwKRv0BiAcemEYEcLR3F9rj4sjUcW4xRiC4)
How I see it implemented
I think there are some quick fixes we can do to improve how this looks:
Those two alone would help it look like this:
I think that's much better already. If we want to go a little further:
In my opinion, the above is much more helpful to spot the differences between the old and the new.
The text was updated successfully, but these errors were encountered: