Skip to content
This repository has been archived by the owner on Dec 15, 2023. It is now read-only.

Commit

Permalink
Dev - minor updates (#73)
Browse files Browse the repository at this point in the history
* Add lint

* Add new libs

* Update libs and add friends tab

* Aggressively hide nonrecent posts

* Update dependencies

* Add php to most recents

* Add full size image downloader

* Fix css cleaner
  • Loading branch information
AllanWang authored Jul 19, 2017
1 parent e4679b1 commit 495b1bd
Show file tree
Hide file tree
Showing 22 changed files with 71 additions and 122 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ script:
- cd $TRAVIS_BUILD_DIR/
- printf "Starting script\n"
- ./gradlew --quiet androidGitVersion
- if [[ "$TRAVIS_BRANCH" == "master" ]]; then ./gradlew publishRelease; else ./gradlew testReleaseUnitTest assembleReleaseTest; fi
- if [[ "$TRAVIS_BRANCH" == "master" ]]; then ./gradlew lintRelease publishRelease; else ./gradlew lintReleaseTest testReleaseUnitTest assembleReleaseTest; fi
notifications:
email: false
slack:
Expand Down
9 changes: 9 additions & 0 deletions app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,15 @@ android {
}
}

lintOptions {
warningsAsErrors true
disable 'LogNotTimber',
'TrustAllX509TrustManager',
'UnusedResources',
'ContentDescription',
'RtlSymmetry'
}

signingConfigs {

def releaseProps = new Properties()
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/assets/css/components/round_icons.compact.css
Original file line number Diff line number Diff line change
@@ -1 +1 @@
#threadlist_rows i.img:not(.profpic), i.img.profpic:not(._1_0m), ._42b6._42b6._42b6, ._5i9c.img, img._2cu2, i.img.l, [id*=threadlist] i.img:not(._1_0m) { border-radius: 50%; }
#threadlist_rows i.img:not(.profpic), i.img.profpic:not(._1_0m), ._42b6._42b6._42b6, ._5i9c.img, img._2cu2, i.img.l, [id*=threadlist] i.img:not(._1_0m) { border-radius: 50% !important; }
2 changes: 1 addition & 1 deletion app/src/main/assets/css/components/round_icons.scss
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
#threadlist_rows i.img:not(.profpic), i.img.profpic:not(._1_0m), ._42b6._42b6._42b6, ._5i9c.img, img._2cu2, i.img.l, [id*=threadlist] i.img:not(._1_0m) {
border-radius: 50%;
border-radius: 50% !important;
}
Original file line number Diff line number Diff line change
Expand Up @@ -40,31 +40,19 @@ class AboutActivity : AboutActivityBase(null, {
val include = arrayOf(
"AboutLibraries",
"AndroidIconics",
"androidslidinguppanel",
"Crashlytics",
"dbflow",
"fastadapter",
"glide",
"Jsoup",
"kau",
"kotterknife",
"materialdialogs",
"materialdrawer"
"materialdrawer",
"subsamplingscaleimageview"
)

/*
* These are great libraries, but either aren't used directly or are too common to be listed
* Give more emphasis on the unique libs!
*/
val exclude = arrayOf(
"GoogleMaterialDesignIcons",
"intellijannotations",
"MaterialDesignIconicIcons",
"MaterialDesignIcons",
"materialize",
"appcompat_v7",
"design",
"recyclerview_v7",
"support_v4"
)
val l = libs.prepareLibraries(this, include, null, false, true)
// l.forEach { KL.d("Lib ${it.definedName}") }
return l
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -166,11 +166,9 @@ class ImageActivity : AppCompatActivity() {
}
}

@Suppress("SIMPLE_DATE_FORMAT")
@Throws(IOException::class)
private fun createImageFile(): File {
// Create an image file name
@SuppressLint("SimpleDateFormat")
val timeStamp = SimpleDateFormat("yyyyMMdd_HHmmss").format(Date())
val imageFileName = "Frost_" + timeStamp + "_"
val storageDir = getExternalFilesDir(Environment.DIRECTORY_PICTURES)
Expand Down Expand Up @@ -264,27 +262,9 @@ internal enum class FabStates(val iicon: IIcon, val iconColor: Int = Prefs.iconC
* If it's in view, give it some animations
*/
fun update(fab: FloatingActionButton) {
if (!fab.isShown) {
fab.setIcon(iicon, color = iconColor)
fab.backgroundTintList = ColorStateList.valueOf(backgroundTint)
} else {
var switched = false
ValueAnimator.ofFloat(1.0f, 0.0f, 1.0f).apply {
duration = 500L
addUpdateListener {
val x = it.animatedValue as Float
val scale = x * 0.3f + 0.7f
fab.scaleX = scale
fab.scaleY = scale
fab.imageAlpha = (x * 255).toInt()
if (it.animatedFraction > 0.5f && !switched) {
switched = true
fab.setIcon(iicon, color = iconColor)
fab.backgroundTintList = ColorStateList.valueOf(backgroundTint)
}
}
start()
}
fab.transition {
setIcon(iicon, color = iconColor)
backgroundTintList = ColorStateList.valueOf(backgroundTint)
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ class MainActivity : BaseActivity(), SearchWebView.SearchContract,
if (BuildConfig.VERSION_CODE > Prefs.versionCode) {
Prefs.versionCode = BuildConfig.VERSION_CODE
if (!BuildConfig.DEBUG) {
showChangelog(R.xml.changelog, Prefs.textColor) { theme() }
showChangelog(R.xml.frost_changelog, Prefs.textColor) { theme() }
frostAnswersCustom("Version") {
putCustomAttribute("Version code", BuildConfig.VERSION_CODE)
putCustomAttribute("Version name", BuildConfig.VERSION_NAME)
Expand Down Expand Up @@ -288,6 +288,7 @@ class MainActivity : BaseActivity(), SearchWebView.SearchContract,
divider()
primaryFrostItem(FbTab.PHOTOS)
primaryFrostItem(FbTab.GROUPS)
primaryFrostItem(FbTab.FRIENDS)
primaryFrostItem(FbTab.PAGES)
divider()
primaryFrostItem(FbTab.EVENTS)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ class SettingsActivity : KPrefActivity(), IabBroadcastReceiver.IabBroadcastListe
items(Support.values().map { string(it.title) })
itemsCallback { _, _, which, _ -> Support.values()[which].sendEmail(this@SettingsActivity) }
}
R.id.action_changelog -> showChangelog(R.xml.changelog, Prefs.textColor) { theme() }
R.id.action_changelog -> showChangelog(R.xml.frost_changelog, Prefs.textColor) { theme() }
else -> return super.onOptionsItemSelected(item)
}
return true
Expand Down
28 changes: 14 additions & 14 deletions app/src/main/kotlin/com/pitchedapps/frost/facebook/FbTab.kt
Original file line number Diff line number Diff line change
Expand Up @@ -11,26 +11,26 @@ import com.pitchedapps.frost.web.FrostWebViewClientMenu
import com.pitchedapps.frost.web.FrostWebViewCore

enum class FbTab(@StringRes val titleId: Int, val icon: IIcon, relativeUrl: String, val webClient: ((webCore: FrostWebViewCore) -> FrostWebViewClient)? = null) {
FEED(R.string.feed, CommunityMaterial.Icon.cmd_newspaper, ""),
FEED_MOST_RECENT(R.string.most_recent, GoogleMaterial.Icon.gmd_history, "?sk=h_chr"),
FEED_TOP_STORIES(R.string.top_stories, GoogleMaterial.Icon.gmd_star, "?sk=h_nor"),
PROFILE(R.string.profile, CommunityMaterial.Icon.cmd_account, "me"),
EVENTS(R.string.events, GoogleMaterial.Icon.gmd_event_note, "events/upcoming"),
FRIENDS(R.string.friends, GoogleMaterial.Icon.gmd_people, "friends/center/requests"),
MESSAGES(R.string.messages, MaterialDesignIconic.Icon.gmi_comments, "messages"),
NOTIFICATIONS(R.string.notifications, MaterialDesignIconic.Icon.gmi_globe, "notifications"),
ACTIVITY_LOG(R.string.activity_log, GoogleMaterial.Icon.gmd_list, "me/allactivity"),
PAGES(R.string.pages, GoogleMaterial.Icon.gmd_flag, "pages"),
GROUPS(R.string.groups, GoogleMaterial.Icon.gmd_group, "groups"),
SAVED(R.string.saved, GoogleMaterial.Icon.gmd_bookmark, "saved"),
BIRTHDAYS(R.string.birthdays, GoogleMaterial.Icon.gmd_cake, "events/birthdays"),
CHAT(R.string.chat, GoogleMaterial.Icon.gmd_chat, "buddylist"),
PHOTOS(R.string.photos, GoogleMaterial.Icon.gmd_photo, "me/photos"),
SETTINGS(R.string.settings, GoogleMaterial.Icon.gmd_settings, "settings"),
EVENTS(R.string.events, GoogleMaterial.Icon.gmd_event_note, "events/upcoming"),
FEED(R.string.feed, CommunityMaterial.Icon.cmd_newspaper, ""),
FEED_MOST_RECENT(R.string.most_recent, GoogleMaterial.Icon.gmd_history, "home.php?sk=h_chr"),
FEED_TOP_STORIES(R.string.top_stories, GoogleMaterial.Icon.gmd_star, "home.php?sk=h_nor"),
FRIENDS(R.string.friends, GoogleMaterial.Icon.gmd_person_add, "friends/center/requests"),
GROUPS(R.string.groups, GoogleMaterial.Icon.gmd_group, "groups"),
MENU(R.string.menu, GoogleMaterial.Icon.gmd_menu, "settings", { FrostWebViewClientMenu(it) }),
MESSAGES(R.string.messages, MaterialDesignIconic.Icon.gmi_comments, "messages"),
NOTES(R.string.notes, CommunityMaterial.Icon.cmd_note, "notes"),
NOTIFICATIONS(R.string.notifications, MaterialDesignIconic.Icon.gmi_globe, "notifications"),
ON_THIS_DAY(R.string.on_this_day, GoogleMaterial.Icon.gmd_today, "onthisday"),
SEARCH(R.string.search_menu_title, GoogleMaterial.Icon.gmd_search, "search")
PAGES(R.string.pages, GoogleMaterial.Icon.gmd_flag, "pages"),
PHOTOS(R.string.photos, GoogleMaterial.Icon.gmd_photo, "me/photos"),
PROFILE(R.string.profile, CommunityMaterial.Icon.cmd_account, "me"),
SAVED(R.string.saved, GoogleMaterial.Icon.gmd_bookmark, "saved"),
SEARCH(R.string.search_menu_title, GoogleMaterial.Icon.gmd_search, "search"),
SETTINGS(R.string.settings, GoogleMaterial.Icon.gmd_settings, "settings"),
;

val url = "$FB_URL_BASE$relativeUrl"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package com.pitchedapps.frost.injectors
import android.graphics.Color
import android.webkit.WebView
import ca.allanwang.kau.utils.*
import com.pitchedapps.frost.utils.L
import com.pitchedapps.frost.utils.Prefs

/**
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
package com.pitchedapps.frost.injectors

import android.webkit.WebView
import com.pitchedapps.frost.utils.L

/**
* Created by Allan Wang on 2017-05-31.
Expand All @@ -16,7 +15,8 @@ enum class CssHider(vararg val items: String) : InjectorContract {
"article[data-store*=sponsor]"
),
PEOPLE_YOU_MAY_KNOW("article._d2r"),
MESSENGER("._s15", "[data-testid=info_panel]", "js_i")
MESSENGER("._s15", "[data-testid=info_panel]", "js_i"),
NON_RECENT("article:not([data-store*=actor_name])")
;

val injector: JsInjector by lazy { JsBuilder().css("${items.joinToString(separator = ",")}{display:none!important}").build() }
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ class JsBuilder {
override fun toString(): String {
val builder = StringBuilder().append("!function(){")
if (css.isNotBlank()) {
val cssMin = css.replace(Regex("\\s+"), "")
val cssMin = css.replace(Regex("[\\r\\n\\t\\f\\v]\\s*"), "")
builder.append("var a=document.createElement('style');a.innerHTML='$cssMin';document.head.appendChild(a);")
}
if (js.isNotBlank())
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ val Context.frostNotification: NotificationCompat.Builder
color = color(R.color.frost_notification_accent)
}

@Suppress("DEPRECATION")
//The update feature is for Android O and seems to still be in beta
fun Notification.frostConfig() = apply {
if (Prefs.notificationVibrate) defaults = defaults or Notification.DEFAULT_VIBRATE
if (Prefs.notificationSound) defaults = defaults or Notification.DEFAULT_SOUND
Expand Down
2 changes: 1 addition & 1 deletion app/src/main/kotlin/com/pitchedapps/frost/utils/iab/IAB.kt
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ fun SettingsActivity.restorePurchases() {
finishRestore(restore, false)
}
getInventory(false, true, reset) {
inv, helper ->
inv, _ ->
val proSku = inv.hasPurchase(FROST_PRO)
Prefs.pro = proSku
L.d("Play Store Restore found: ${Prefs.pro}")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@ open class FrostWebViewClient(val webCore: FrostWebViewCore) : BaseWebViewClient
internal fun injectAndFinish() {
L.d("Page finished reveal")
webCore.jsInject(CssHider.HEADER,
CssHider.NON_RECENT.maybe(webCore.url.contains("?sk=h_chr")),
Prefs.themeInjector,
callback = {
refreshObservable.onNext(false)
Expand All @@ -104,15 +105,25 @@ open class FrostWebViewClient(val webCore: FrostWebViewCore) : BaseWebViewClient
* returns true to override the url
*/
private fun launchRequest(request: WebResourceRequest): Boolean {
L.d("Launching ${request.url}")
L.d("Launching Url", request.url.toString())
webCore.context.launchWebOverlay(request.url.toString())
return true
}

private fun launchImage(request: WebResourceRequest, text: String? = null): Boolean {
L.d("Launching Image", request.url.toString())
webCore.context.launchImageActivity(request.url.toString(), text)
if (webCore.canGoBack()) webCore.goBack()
return true
}

override fun shouldOverrideUrlLoading(view: WebView, request: WebResourceRequest): Boolean {
L.i("Url Loading ${request.url}")
val path = request.url.path ?: return super.shouldOverrideUrlLoading(view, request)
L.v("Url Loading Path $path")
if (path.startsWith("/composer/")) return launchRequest(request)
if (request.url.toString().contains("scontent-sea1-1.xx.fbcdn.net") && (path.endsWith(".jpg") || path.endsWith(".png")))
return launchImage(request)
return super.shouldOverrideUrlLoading(view, request)
}

Expand Down
1 change: 0 additions & 1 deletion app/src/main/res/layout/item_keyword.xml
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,4 @@
android:background="?android:attr/selectableItemBackground"
android:scaleType="center" />


</LinearLayout>
27 changes: 0 additions & 27 deletions app/src/main/res/layout/web_context_menu.xml

This file was deleted.

23 changes: 0 additions & 23 deletions app/src/main/res/transition/web_context_enter.xml

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -9,17 +9,18 @@
-->

<version title="Beta Updates" />
<item text="Fix notification duplicates" />
<item text="Fix long pressing album images" />
<item text="" />
<item text="" />
<item text="" />

<item text="Add friend request tab in nav bar" />
<item text="Aggressively filter nonrecent posts in recents mode" />
<item text="Add download option for full sized images" />
<item text="" />

<version title="v1.4" />
<item text="Update IAB helper" />
<item text="Create image viewing and downloading; long press any image!" />
<item text="Start filtering out unnecessary loads" />
<item text="Fix notification duplicates" />
<item text="Fix long pressing album images" />

<version title="v1.3" />
<item text="Create toggle for notifications only from primary account" />
Expand Down
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ task clean(type: Delete) {
}

task generateChangelogMd() {
def parsedProjectXml = (new XmlParser()).parse("$project.rootDir/app/src/main/res/xml/changelog.xml")
def parsedProjectXml = (new XmlParser()).parse("$project.rootDir/app/src/main/res/xml/frost_changelog.xml")
def sw = new StringWriter()
sw.append("# Changelog\n")
parsedProjectXml.depthFirst().each {
Expand Down
7 changes: 7 additions & 0 deletions docs/Changelog.md
Original file line number Diff line number Diff line change
@@ -1,9 +1,16 @@
# Changelog

## Beta Updates
* Add friend request tab in nav bar
* Aggressively filter nonrecent posts in recents mode
* Add download option for full sized images

## v1.4
* Update IAB helper
* Create image viewing and downloading; long press any image!
* Start filtering out unnecessary loads
* Fix notification duplicates
* Fix long pressing album images

## v1.3
* Create toggle for notifications only from primary account
Expand Down
4 changes: 2 additions & 2 deletions gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ MIN_SDK=21
TARGET_SDK=26
BUILD_TOOLS=26.0.0

KAU=eddaded
KAU=v2.1
KOTLIN=1.1.3-2
CRASHLYTICS=2.6.8
DBFLOW=4.0.4
DBFLOW=4.0.5
GLIDE=4.0.0-RC1
IICON_COMMUNITY=1.9.32.2
IICON_MATERIAL=2.2.0.3
Expand Down

0 comments on commit 495b1bd

Please sign in to comment.