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

Add a section to allow the user to save his/her favourite skills #322 #573

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sangu80
Copy link

@Sangu80 Sangu80 commented Aug 8, 2020

   struct landmarkList : View {
       @state var showFavoritesOnly = false

       var body : some View{
           NavigationView{
               List(landmarkData) { landmark in
                   if !self,showFavoritesOnly || landmark.isFavorite {
               NavigaitonLink(destination: LandmarkDetail(landmark: landmark))
                    Landmarkrow(landmark: landmark)

                }
           }
           .navigationBarTitle(Text("Landmarks"))
       }
   }

Fixes #[Add issue number here. If you do not solve the issue entirely, please change the message e.g. "First steps for issues #IssueNumber]

Changes: [Add here what changes were made in this issue and if possible provide links.]

Screenshots for the change:

…asia#322

       struct landmarkList : View {
           @State var showFavoritesOnly = false

           var body : some View{
               NavigationView{
                   List(landmarkData) { landmark in
                       if !self,showFavoritesOnly || landmark.isFavorite {
                   NavigaitonLink(destination: LandmarkDetail(landmark: landmark))
                        Landmarkrow(landmark: landmark)

                    }
               }
               .navigationBarTitle(Text("Landmarks"))
           }
       }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant