-
Notifications
You must be signed in to change notification settings - Fork 130
osharon: Detective Adventures #387
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
base: gh-pages
Are you sure you want to change the base?
osharon: Detective Adventures #387
Conversation
lhancock-scottlogic
left a comment
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.
Looks great, just a few minor tweaks suggested! Also, have you spoken to someone on your project about including the screenshots in this post? I think it should be fine but always good to clear it with someone if it's a client system you're showing!
|
|
||
| Another suggestion we have considered was to use HTML-based summary/details elements to show/hide the secondary actions, thus eliminating the need for JavaScript. Unfortunately, the design team was not in favour of this solution due to time constraints on their part. | ||
|
|
||
| There is a key takeaway about the [difference](https://www.enonic.com/blog/what-is-the-difference-between-server-side-and-client-side) between form-based applications (server-side rendering for that matter) and web-based applications. The former is extremely limited in its capabilities, and I can see why designers would find it too restricting. Personally, I find that it forces a certain simplicity that is healthy for code maintenance as well as the user’s mental model. |
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.
Could possibly include an example here of the simplicity you're referring to, or just expand a little bit on this part?
|
|
||
| There is a key takeaway about the [difference](https://www.enonic.com/blog/what-is-the-difference-between-server-side-and-client-side) between form-based applications (server-side rendering for that matter) and web-based applications. The former is extremely limited in its capabilities, and I can see why designers would find it too restricting. Personally, I find that it forces a certain simplicity that is healthy for code maintenance as well as the user’s mental model. | ||
|
|
||
| The landscape of software development is constantly changing, with modern technologies, frameworks, and methodologies. This, as I have mentioned in the past, can be quite overwhelming for junior developers. I hope that this experience I have shared gives a better view on the type of problems we are dealing with. As developers, we are required to understand the client needs and produce the best software, regardless of the tech stack. |
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.
This sounds like a good conclusion, you may want to add a Conclusion sub title as I often see in our other blog posts :)
| @@ -0,0 +1,46 @@ | |||
| --- | |||
| title: Detective Adventures | |||
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 think adding a few descriptive words to the blog's title may make it more eye-catching? Just a few words after Detective Adventures: that would help the reader to understand exactly what the article is about straight away :)
Co-authored-by: lhancock-scottlogic <110816543+lhancock-scottlogic@users.noreply.github.com>
f3f59b4 to
73f899e
Compare
73f899e to
8a73501
Compare
Thank you @lhancock-scottlogic , I fixed the issues mentioned and updated the date; |
Please add a direct link to your post here:
Detective Adventures
Have you (please tick each box to show completion):
npm installfollowed bynpx mdspell "**/{FILE_NAME}.md" --en-gb -a -n -x -tif that's your thing)Posts are reviewed / approved by your Regional Tech Lead.