Reinstate "Monitoring and Evaluation content"" #6

Merged
awolski merged 1 commits from peml into main 8 months ago
awolski commented 8 months ago
Owner

Website with changes: https://b5c17133.yvant-coop-github-io.pages.dev/

I had to make a few decisions.

  • When we only have the technology content the technology image sits nicely beside the 'Overview' text.
  • When the PEML content is introduced the Technology/PEML content gets pushed down by the technology image.
  • The overview text is very general, and I think more focused on technology. It made sense to me to move this into the Technology collapsible content. Maybe we need a similar section in the collapsible PEML content.
  • This leaves straight-to-the-point sections for Technology and Monitoring & Evaluation. But oesn't leave a logical place for an 'Overview' image in this section without pushing the content all the way down.
  • I think it looks clean and to the point without the image, but thoughts welcome
Website with changes: https://b5c17133.yvant-coop-github-io.pages.dev/ I had to make a few decisions. * When we only have the technology content the technology image sits nicely beside the 'Overview' text. * When the PEML content is introduced the Technology/PEML content gets pushed down by the technology image. * The overview text is very general, and I think more focused on technology. It made sense to me to move this into the Technology collapsible content. Maybe we need a similar section in the collapsible PEML content. * This leaves straight-to-the-point sections for Technology and Monitoring & Evaluation. But oesn't leave a logical place for an 'Overview' image in this section without pushing the content all the way down. * I think it looks clean and to the point without the image, but thoughts welcome
awolski added 1 commit 8 months ago
awolski requested review from iperdomo 8 months ago
awolski requested review from ethelnmc 8 months ago
awolski was assigned by iperdomo 8 months ago
awolski merged commit 388d031c4c into main 8 months ago
awolski deleted branch peml 8 months ago

Reviewers

iperdomo was requested for review 8 months ago
ethelnmc was requested for review 8 months ago
The pull request has been merged as 388d031c4c.
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.