I don't find bracketing especially helpful as it "hides" some of the source citation information. I think people are really in two camps about this and feel strongly about whichever one they're in.
I do agree that including full citations to the exact record is useful. Good citations will last over time: When that link goes bad or a website doesn't exist any more, a good citation will still tell you exactly where you might find the original record.
FamilySearch's pre-written citations are slightly better at this than Ancestry, but not always. On Ancestry sometimes you really have to dig to find out where they got the information. They're a repository of copies, not the original record holders (though they seem to like people to think otherwise ).
For the example that Patricia used, the reference to FS's internal database that they lead with doesn't really tell you much, but adding that it's an index rather than an original record could tell us a lot.
Marriage record (index) of I.N. Ballow and Jessie Gipson, 01 Jan 1908, https://www.familysearch.org/ark:/61903/1:1:F61H-CM4, FHL microfilm 1,020,947, citing Rusk, Texas, United States County Courthouse.
Knowing that it's an index tells us that there might be another record collection that has the originals, that there might be additional information in the originals, and that there might be transcription errors in this record. (I don't mean that to sound like a criticism for using an index or transcription. Sometimes it's all we've got.)
I also love that you did in-line citations! I personally find it really helpful to know where different bits of information came from. In-line citations is some "advanced WikiTree'ing!"
Over all, really nicely done!