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

[show-all-data] Make the class fieldCalculated more visible by increasing the font-weight #680

Closed
lrlip opened this issue Dec 12, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@lrlip
Copy link

lrlip commented Dec 12, 2024

Is your feature request related to a problem? Please describe.
The current fieldCalculated class does not clearly differentiate between calculated fields and native Salesforce object fields. This lack of visual or structural distinction makes it difficult to identify which fields are dependent and which are part of the original Salesforce schema, leading to confusion and inefficiencies during use.

Describe the solution you'd like
Introduce enhancements to the fieldCalculated class to improve clarity:

  1. Apply bold text or a distinct color to make calculated fields stand out visually.
  2. Add an extra column to explicitly indicate whether a field is a formula or a lookup.

Describe alternatives you've considered

  1. Alternatively, append a suffix, such as [F], to fields that are formulas, providing immediate recognition of their type.

Additional context

@lrlip lrlip added the enhancement New feature or request label Dec 12, 2024
@lrlip lrlip changed the title [Show-All-Data] Make the class fieldCalculated more visible be increasing the font-weight [show-all-data] Make the class fieldCalculated more visible be increasing the font-weight Dec 12, 2024
@lrlip lrlip changed the title [show-all-data] Make the class fieldCalculated more visible be increasing the font-weight [show-all-data] Make the class fieldCalculated more visible by increasing the font-weight Dec 12, 2024
@tprouvot
Copy link
Owner

Today the difference is represented by * char concatenated to the field type

@lrlip
Copy link
Author

lrlip commented Dec 13, 2024

Yes indeed. In Salesforce, a * indicates a required field, while in the extension, it marks a calculated field, and required fields are written out explicitly. Aligning these conventions provides a better user experience.

tprouvot pushed a commit that referenced this issue Dec 17, 2024
## Add the `calculated` type for formula fields in the show-all-data 

##
[#680](#680)

## Checklist before requesting a review
- [x] I have read and understand the [Contributions
section](https://github.com/tprouvot/Salesforce-Inspector-reloaded#contributions)
- [x] Target branch is releaseCandidate and not master
- [x] I have performed a self-review of my code
- [x] I ran the [unit
tests](https://github.com/tprouvot/Salesforce-Inspector-reloaded#unit-tests)
and my PR does not break any tests
- [x] I documented the changes I've made on the
[CHANGES.md](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/CHANGES.md)
and followed actual conventions
- [ ] I added a new section on
[how-to.md](https://github.com/tprouvot/Salesforce-Inspector-reloaded/blob/master/docs/how-to.md)
(optional)
@lrlip lrlip closed this as completed Dec 17, 2024
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
Development

No branches or pull requests

2 participants