Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Commit d7d4c77

Browse files
committed
Implement review feedback
1 parent a832977 commit d7d4c77

File tree

5 files changed

+12
-47
lines changed

5 files changed

+12
-47
lines changed
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
## [0.1.0] - 2017-05-05
2+
3+
* Initial Open Source release.

packages/shared-preferences/example/android/app/src/main/AndroidManifest.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
2-
package="com.yourcompany.shared_preferences_example"
2+
package="io.flutter.plugins.shared_preferences_example"
33
android:versionCode="1"
44
android:versionName="0.0.1">
55

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package com.yourcompany.shared_preferences_example;
1+
package io.flutter.plugins.shared_preferences_example;
22

33
import android.os.Bundle;
44
import io.flutter.app.FlutterActivity;

packages/shared-preferences/example/lib/main.dart

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
// Copyright 2017 The Chromium Authors. All rights reserved.
2+
// Use of this source code is governed by a BSD-style license that can be
3+
// found in the LICENSE file.
4+
15
import 'dart:async';
26

37
import 'package:flutter/material.dart';
@@ -39,14 +43,14 @@ class SharedPreferencesDemoState extends State<SharedPreferencesDemo> {
3943
Widget build(BuildContext context) {
4044
return new Scaffold(
4145
appBar: new AppBar(
42-
title: new Text("SharedPreferences Demo"),
46+
title: const Text("SharedPreferences Demo"),
4347
),
4448
body: new Center(
4549
child: new FutureBuilder(
4650
future: _prefs,
4751
builder: (BuildContext context, AsyncSnapshot<SharedPreferences> snapshot) {
48-
if (snapshot.connectionState != ConnectionState.done)
49-
return new Text('Loading...');
52+
if (snapshot.connectionState == ConnectionState.waiting)
53+
return const Text('Loading...');
5054
int counter = snapshot.requireData.getInt('counter') ?? 0;
5155
return new Text(
5256
'Button tapped $counter time${ counter == 1 ? '' : 's' }.\n\n'

packages/shared-preferences/example/pubspec.yaml

Lines changed: 0 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -7,47 +7,5 @@ dependencies:
77
shared_preferences:
88
path: ../
99

10-
# For information on the generic Dart part of this file, see the
11-
# following page: https://www.dartlang.org/tools/pub/pubspec
12-
13-
# The following section is specific to Flutter.
1410
flutter:
15-
16-
# The following line ensures that the Material Icons font is
17-
# included with your application, so that you can use the icons in
18-
# the Icons class.
1911
uses-material-design: true
20-
21-
# To add assets to your application, add an assets section here, in
22-
# this "flutter" section, as in:
23-
# assets:
24-
# - images/a_dot_burr.jpeg
25-
# - images/a_dot_ham.jpeg
26-
27-
# To add assets from package dependencies, first ensure the asset
28-
# is in the lib/ directory of the dependency. Then,
29-
# refer to the asset with a path prefixed with
30-
# `packages/PACKAGE_NAME/`. Note: the `lib/` is implied, do not
31-
# include `lib/` in the asset path.
32-
#
33-
# Here is an example:
34-
#
35-
# assets:
36-
# - packages/PACKAGE_NAME/path/to/asset
37-
38-
# To add custom fonts to your application, add a fonts section here,
39-
# in this "flutter" section. Each entry in this list should have a
40-
# "family" key with the font family name, and a "fonts" key with a
41-
# list giving the asset and other descriptors for the font. For
42-
# example:
43-
# fonts:
44-
# - family: Schyler
45-
# fonts:
46-
# - asset: fonts/Schyler-Regular.ttf
47-
# - asset: fonts/Schyler-Italic.ttf
48-
# style: italic
49-
# - family: Trajan Pro
50-
# fonts:
51-
# - asset: fonts/TrajanPro.ttf
52-
# - asset: fonts/TrajanPro_Bold.ttf
53-
# weight: 700

0 commit comments

Comments
 (0)