Skip to content

Commit

Permalink
chore(frontend/lint): Fix lint issues
Browse files Browse the repository at this point in the history
  • Loading branch information
Grohden committed Aug 5, 2020
1 parent 0194579 commit 4b7cfc6
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 62 deletions.
2 changes: 1 addition & 1 deletion frontend/lib/ui/pages/home/home_page.dart
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import 'package:flutter/material.dart';
import 'package:get/get.dart';

import '../../templates/two_slot_container.dart';
import '../../templates/three_slot_container.dart';
import '../starred/starred_page.dart';
import '../tags/tags_page.dart';
import 'widgets/sidebar.dart';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,12 @@ import 'dart:html';
import 'package:dio/dio.dart';
import 'package:flutter/material.dart';
import 'package:get/get.dart';
import 'package:repo_tagger/ui/utils/browser.dart';

import '../../../api/tagger/repository_tagger_client.dart';
import '../../../router.dart';
import '../../molecules/detail_chip.dart';
import '../../molecules/load_page_error.dart';
import '../../utils/browser.dart';
import 'widgets/readme_container.dart';
import 'widgets/tags_container.dart';

Expand Down
2 changes: 1 addition & 1 deletion frontend/lib/ui/pages/splash/splash_page.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ import 'dart:async';

import 'package:flutter/material.dart';
import 'package:get/get.dart';
import 'package:repo_tagger/api/tagger/repository_tagger_client.dart';

import '../../../api/tagger/repository_tagger_client.dart';
import '../../../router.dart';

/// Initial page that redirects the user
Expand Down
4 changes: 1 addition & 3 deletions frontend/lib/ui/pages/starred/starred_page.dart
Original file line number Diff line number Diff line change
@@ -1,13 +1,11 @@
import 'package:flutter/material.dart';
import 'package:get/get.dart';
import 'package:repo_tagger/ui/templates/repository_list.dart';

import '../../../api/tagger/repository_tagger_client.dart';
import '../../../router.dart';
import '../../molecules/detail_chip.dart';
import '../../molecules/load_page_error.dart';
import '../../molecules/page_title.dart';
import '../../templates/page_body.dart';
import '../../templates/repository_list.dart';

part 'starred_bindings.dart';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
import 'package:flutter/material.dart';
import 'package:get/get.dart';
import 'package:repo_tagger/ui/templates/repository_list.dart';

import '../../../api/tagger/repository_tagger_client.dart';
import '../../../router.dart';
import '../../molecules/load_page_error.dart';
import '../../molecules/page_title.dart';
import '../../templates/page_body.dart';
import '../../templates/repository_list.dart';

part 'tag_repositories_bindings.dart';

Expand Down Expand Up @@ -40,52 +39,4 @@ class TagRepositoriesPage extends GetView<TagRepositoriesController> {
items: controller.repositories.value,
);
}

Widget _buildList(List<SimpleRepository> items, TextTheme style) {
return ListView.separated(
itemCount: items.length,
padding: const EdgeInsets.all(kStandardPadding),
separatorBuilder: (_, __) => const Divider(thickness: 1),
itemBuilder: (context, index) {
final item = items[index];

return ListTile(
onTap: () => controller.openRepo(item),
title: Text(item.name, style: style.headline6),
subtitle: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
if (item.description?.isNotEmpty == true)
Text(item.description ?? ''),
_buildChipDetails(item),
],
),
);
});
}

Widget _buildChipDetails(SimpleRepository item) {
const spacing = SizedBox(width: 4);

return Row(
children: [
if (item.language?.isNotEmpty == true)
Chip(
visualDensity: VisualDensity.compact,
label: Text(item.language.toString()),
),
spacing,
Chip(
visualDensity: VisualDensity.compact,
label: Row(
children: [
const Icon(Icons.star_border, size: 14),
spacing,
Text(item.stargazersCount.toString()),
],
),
),
],
);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -20,17 +20,13 @@ class ThreeSlotContainer extends StatelessWidget {

@override
Widget build(BuildContext context) {
final theme = Theme.of(context);

return Row(
mainAxisSize: MainAxisSize.max,
crossAxisAlignment: CrossAxisAlignment.start,
children: [
if (leftSlot != null)
SizedBox(width: leftWidth, child: leftSlot),
if (leftSlot != null) SizedBox(width: leftWidth, child: leftSlot),
Expanded(child: centralSlot),
if (rightSlot != null)
SizedBox(width: rightWidth, child: rightSlot)
if (rightSlot != null) SizedBox(width: rightWidth, child: rightSlot)
],
);
}
Expand Down

0 comments on commit 4b7cfc6

Please sign in to comment.