Custom favicon is not displayed

[ghost]

This block here seems wrong. It is doing the same thing in both cases.

dashR/R/dash.R

Lines 1897 to 1905 in d281e70

	# create tag for favicon, if present
	# other_files_map[names(other_files_map) %in% "/favicon.ico"]
	if ("/favicon.ico" %in% names(private$asset_map$other)) {
	favicon_url <- sprintf('\"%s_favicon.ico\"', self$config$requests_pathname_prefix)
	favicon <- sprintf("<link href=%s rel=\"icon\" type=\"image/x-icon\">", favicon_url)
	} else {
	favicon_url <- sprintf('\"%s_favicon.ico\"', self$config$requests_pathname_prefix)
	favicon <- sprintf("<link href=%s rel=\"icon\" type=\"image/x-icon\">", favicon_url)
	}

[ghost]

Also see #297. A workaround:

app_title <- "My app"
app_custom_index <- local({
  template <- app$.__enclos_env__$private$template_index[[1]]
  template <- sub("{%favicon%}",
                  "<link href=\"assets/favicon.svg\" rel=\"icon\" type=\"image/x-icon\">",
                  template, fixed = TRUE)
  template <- sub("{%title%}", app_title, template, fixed = TRUE)
  return(template)
})
app$index_string(app_custom_index)

[rpkyle]

This block here seems wrong. It is doing the same thing in both cases.

dashR/R/dash.R

Lines 1897 to 1905 in d281e70

	# create tag for favicon, if present
	# other_files_map[names(other_files_map) %in% "/favicon.ico"]
	if ("/favicon.ico" %in% names(private$asset_map$other)) {
	favicon_url <- sprintf('\"%s_favicon.ico\"', self$config$requests_pathname_prefix)
	favicon <- sprintf("<link href=%s rel=\"icon\" type=\"image/x-icon\">", favicon_url)
	} else {
	favicon_url <- sprintf('\"%s_favicon.ico\"', self$config$requests_pathname_prefix)
	favicon <- sprintf("<link href=%s rel=\"icon\" type=\"image/x-icon\">", favicon_url)
	}

Anecdotally, I suspect you are correct @sid-bagaria. It's been a long time since I worked on this project, but if I recall the original else set favicon to NULL, and this was later modified to store an empty string if no valid favicon was passed.

I'm not sure whether this is copypasta, but it seems possible.