Completed basic design of navbar, sidebar, search tool, accordions, help and feedback #2

Merged
dtookey merged 24 commits from RebeccaBranch into master 5 years ago
Collaborator

I've completed several goals for the basic design. This seems like a good point to send what I have in preparation for the next phase of work.

I've completed several goals for the basic design. This seems like a good point to send what I have in preparation for the next phase of work.
bhollis added 23 commits 5 years ago
bhollis added 1 commit 5 years ago
Owner

I'm going to approve this, but I have comments about a few things.

  • Are we sure we want a feedback button?
  • In src/assets/pages/styles/resources-landing-page.scss, are you sure that you want inline svgs instead of keeping them in their own file and referencing them with an <img> element? (line 133)
I'm going to approve this, but I have comments about a few things. * Are we sure we want a feedback button? * In src/assets/pages/styles/resources-landing-page.scss, are you sure that you want inline svgs instead of keeping them in their own file and referencing them with an `<img>` element? (line 133)
dtookey approved these changes 5 years ago
dtookey closed this pull request 5 years ago
dtookey reopened this pull request 5 years ago
dtookey approved these changes 5 years ago
dtookey left a comment
Owner

Huh, it didn't merge on the first approval

Huh, it didn't merge on the first approval
dtookey changed title from WIP: Completed basic design of navbar, sidebar, search tool, accordions, help and feedback to Completed basic design of navbar, sidebar, search tool, accordions, help and feedback 5 years ago
dtookey merged commit dd4c349321 into master 5 years ago

Reviewers

dtookey approved these changes 5 years ago
The pull request has been merged as dd4c349321.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b RebeccaBranch master
git pull origin RebeccaBranch

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff RebeccaBranch
git push origin master
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: dtookey/Engine_Rebuild#2
Loading…
There is no content yet.